Export as DXF with layer selection improvement (all, visible, by name)

[Yann Papouin]

Hi,

I have added export by name support to dxf_outline extension.
I've removed previously added "visibleLayers" option that is now embedded in the new layer_option.
Now you can choose to export all layers, only visible ones or by name match (case insensitive and use comma ',' as separator)

See files attached

[Alvin Penner (apenner)]

thanks for the contribution!
I have tested it and confirmed that it behaves correctly if one uses the option 'By name match', and if one specifies a layer name that actually exists.

However, there are two cases where it does not behave as expected.
case 1, use the option 'By name match', and leave the layer name blank. In this case it sends all layers to output, when I would have expected none.
case 2, deliberately misspell a layer name so no match is found. In this case I get the attached error message, saying it failed to save the file.

In both of these cases, I think it would be preferable to raise an information window, using something like inkex.errormsg, and indicate to the user that no layer name match was found, and ask them to try again.

[Yann Papouin (yann-papouin)]

Ok that should not be difficult, I'm looking into this.
Thanks for your review.

[Yann Papouin (yann-papouin)]

Updated!

- case 1, an error message is now displayed if 'Layer match name' field is empty, and the export process is canceled.
- case 2, a warning message is displayed if no match is found for a layer name, but the export will process successfully

[Alvin Penner (apenner)]

thanks, tested and committed to rev 12273