Dear Jerermy! On 12/10/12 16:34, Jeremy Sanders wrote: > On 10/12/2012 03:29 AM, Dr. Volker Jaenisch wrote: > >> While patching the code I feel bad. It is a hack but no solution. I would >> like to contribute to build a modular Import-dialog-Framework that >> is capable to assimilate the new Code I will bring in. That will be no great >> deal, just moving a few classes into a base module and organize >> that the qt-resources are >> bound to an external data structure. > > This sounds a great idea. Thanks for your offer of making the dialog more > extensible. I designed the import plugin to be able to do this, but > it's not flexible enough for complex data formats. Thank you very much for your welcome. > > Please go ahead and have a look at doing this. If you want to discuss any > designs, we can do it here or offline. There are a few places which > need to be modified in order to add a new data format - in the > document.operations, document.importparams, documents.linkedfile, the dialog > itself... If you can clean this up to make it possible to add new formats in > an easy way, please do. My first guess would be to seperate the base-classes ImportParamsBase, ImportDialogBase etc. from the concrete ones. The Classes needed for the import, and representation of a certain Fileformat then should be more located. E.G. there may be a folder fileformats with a file for each fileformat, containing the 'incredients' of the fileformat. Added a minimal sort of registry for the fileformats and we will end up with sort of a plugin system for the fileformats.
This is pure guessing. A had only given a glance to your code and may have orverseen some complexity that renders this simple approach useless. I will give it a try and do some refactoring and wait for the problems to come. Best regards Volker -- Dr. Volker Jaenisch Geschäftsführer Inqbus GmbH & Co. KG Softwareentwicklung, Consulting & Hosting Karl-Heine-Straße 99 | 04229 Leipzig | Deutschland Telefon: +49 341 989758-54 Fax: +49 341 989758-79 E-Mail: [email protected] Web: http://inqbus.de/ Persönlich haftende Gesellschafterin: Inqbus Management GmbH (Amtsgericht Leipzig, HRB 27350) Vertretungsberechtigte Geschäftsführer: Maik Derstappen, Dr. Volker Jaenisch, Thomas Massmann, Markus Zapke-Gründemann Registergericht: Amtsgericht Leipzig Registernummer: HRA 16424 Umsatzsteuer-Identifikationsnummer: DE278744671 _______________________________________________ Veusz-discuss mailing list [email protected] https://mail.gna.org/listinfo/veusz-discuss
