hexylena / galaxyxml

XML Generation libraries for Galaxy Tool/ToolDeps XML
Apache License 2.0
6 stars 5 forks source link

use copy.deepcopy to prevent duplication of information when using export() several times #10

Closed khillion closed 7 years ago

khillion commented 7 years ago

Hi Eric,

I realized that information written in the XML was duplicated when the export() method from the Tool class was used several times. Here I propose to use the copy.deepcopy() method to prevent this behaviour.

Thank you, Kenzo

hexylena commented 7 years ago

I'd never tried calling export multiple times. Interesting. I feel like there might be a more technically pretty solution than copy.deepcopy on self, but the tests pass so this is fine.

khillion commented 7 years ago

Thank you, I also think there must be another way to do it.