[clam-devel] Changes to NetworkEditor
Natanael Olaiz
nolaiz at gmail.com
Wed Jun 30 14:18:53 PDT 2010
Hi,
El 06/30/2010 09:13 PM, Xavier Serra escribió:
> On Jun 30, 2010, at 5:01 PM, David García Garzón wrote:
>
>> On Monday 21 June 2010 17:14:57 Xavier Serra wrote:
>>
>>> Hi guys,
>>> I'm Xavi Serra, an student at UPF doing his final project for CLAM with
>>> David.
>>>
>>> We intend to extend the funcionality of the NetworkEditor. The first
>>> thing we are doing is to detect when a network file is outdated and
>>> automatically throw clamRefactor (asking permission to the user to do
>>> so).
>>>
>>> [snip]
>>>
>>> Xavier Serra
>>>
>> Committed after some refactorings we did yesterday together. Now a dialog is
>> shown on version mismatch and in the case of a network of an older version it
>> gives you the option to upgrade the network by using clamrefactor before
>> loading it. The upgrade is still dummy, but Xavi is already working on it.
>>
>> Xavi, some screenshots would be helpful so that other developers can do
>> comments on usability aspects. Could you put them on the wiki?
>>
>>
> Done.
>
> It would be great too if you guys could give some feedback about usability issues. I think the messages are now, after David's modification, more clear but I'd prefer more opinions.
>
>
The messages are clear to me.
But I have some questions:
1- The clamVersion attribute did always existed on the clam network
files? (I cannot found it in the CHANGES log) If not, there are
probably no clam networks so much outdated being used... but what would
happens if it doesn't exist? From
http://doc.trolltech.com/4.5/qxmlquery.html#evaluateTo-4 : "If output is
null behavior is undefined."
2- What would be the behavior in case of different versions? I think
that checking the version number is OK as a first test of possible
conflict between releases, but the needed changes and the definitely
tests should be done looking in a migrationScript file or something like
that. How we would deal with the different migrationScripts that we have
in the core and different plugins? We would include all of them in the
NetworkEditor to make such a test?
3- David: do you think that it would be difficult to put the interface
to check the network version on the CLAM::XMLStorage class? I know that
this improvement and interface to solve the version conflict is
inherent to the NetworkEditor, but CLAM::XMLStorage would not be the
ideal place to just check the version? Maybe returning diferent error
codes on the Restore...
If we maintain updated the migrationScript(s) file(s), this would be a
very nice tool. :)
Best regards,
Natanael.
> Xavi Serra
>
> _______________________________________________
> clam-devel mailing list
> clam-devel at lists.clam-project.org
> http://lists.clam-project.org/listinfo.cgi/clam-devel-clam-project.org
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.clam-project.org/pipermail/clam-devel-clam-project.org/attachments/20100630/baf8af67/attachment-0004.htm>
More information about the clam-devel
mailing list