[clam-devel] Changes to NetworkEditor

Xavier Serra xvr.serra at gmail.com
Wed Jun 30 15:17:56 PDT 2010


On Jun 30, 2010, at 11:18 PM, Natanael Olaiz wrote:

> 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." 
> 

It didn't. Files without the attribute are treated as older file and thus, need conversion. David told me these files are pre-1.2 or so.

> 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?
> 

When NetworkEditor detects that a file is older it will throw clamRefactor. The idea is to have a migrationScript file with the steps needed to go from version to version and use that when calling clamRefactor

> 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
>> 
>>   
> 
> _______________________________________________
> 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/20100701/6e0c95c8/attachment.html>


More information about the clam-devel mailing list