[clam-devel] Changes to NetworkEditor

[Natanael Olaiz]

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-0003.htm>