[jira] Updated: (JCR-556) Refactoring of the BackupTool

2007-02-22 Thread Nicolas Toper (JIRA)
[ https://issues.apache.org/jira/browse/JCR-556?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Nicolas Toper updated JCR-556: -- Attachment: VisitorPattern22022007.patch Hello, Thanks for your comment. VisitorPattern22022007.patch

[jira] Updated: (JCR-556) Refactoring of the BackupTool

2007-02-22 Thread Nicolas Toper (JIRA)
[ https://issues.apache.org/jira/browse/JCR-556?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Nicolas Toper updated JCR-556: -- Attachment: VisitorPattern22022007-2.patch VisitorPattern22022007-2.patch is checkstyle compliant.

[jira] Commented: (JCR-556) Refactoring of the BackupTool

2007-02-22 Thread Nicolas Toper (JIRA)
[ https://issues.apache.org/jira/browse/JCR-556?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12475018 ] Nicolas Toper commented on JCR-556: --- Stefan, You are right. It is more elegant. I'll update the code and rewrite

[jira] Updated: (JCR-556) Refactoring of the BackupTool

2007-02-22 Thread Nicolas Toper (JIRA)
[ https://issues.apache.org/jira/browse/JCR-556?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Nicolas Toper updated JCR-556: -- Attachment: VisitorPattern22022007-3.patch VisitorPattern22022007-3.patch implements all the required

[jira] Commented: (JCR-556) Refactoring of the BackupTool

2007-02-22 Thread Nicolas Toper (JIRA)
[ https://issues.apache.org/jira/browse/JCR-556?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12475049 ] Nicolas Toper commented on JCR-556: --- Yes my mistake. Why don't you want to check for initialization of the

[jira] Updated: (JCR-556) Refactoring of the BackupTool

2007-02-22 Thread Nicolas Toper (JIRA)
[ https://issues.apache.org/jira/browse/JCR-556?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Nicolas Toper updated JCR-556: -- Attachment: Visitor2202007-final.patch Hi, Here is the patch with Stefan's suggestion and tested it. I

[jira] Updated: (JCR-556) Refactoring of the BackupTool

2007-02-20 Thread Nicolas Toper (JIRA)
[ https://issues.apache.org/jira/browse/JCR-556?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Nicolas Toper updated JCR-556: -- Attachment: VisitorPattern200207.patch Hi, I still haven't quit working on the backup tool. Thanks to

[jira] Updated: (JCR-535) Ignore root node when importing through sysView

2006-11-19 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-535?page=all ] Nicolas Toper updated JCR-535: -- Attachment: WorkspaceImporterRootNodeTest.patch Hi, Here is a test code demonstrating the issue and that it applies only to REPLACE_EXISTING behaviour. Nicolas

[jira] Commented: (JCR-535) Ignore root node when importing through sysView

2006-10-26 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-535?page=comments#action_12444926 ] Nicolas Toper commented on JCR-535: --- Hi, About this issue, can I assume there are no concurrent operations? I am deleting all subnodes under root except

[jira] Commented: (JCR-535) Ignore root node when importing through sysView

2006-10-24 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-535?page=comments#action_12444311 ] Nicolas Toper commented on JCR-535: --- Sure. Let me write it. I'll post it this week. Ignore root node when importing through sysView

[jira] Commented: (JCR-556) Refactoring of the BackupTool

2006-10-23 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-556?page=comments#action_12444079 ] Nicolas Toper commented on JCR-556: --- Hi, For those wondering, what is the status of the backup tool. It is working, but to make it really useful, Jukka and I are

[jira] Commented: (JCR-535) Ignore root node when importing through sysView

2006-10-23 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-535?page=comments#action_12444114 ] Nicolas Toper commented on JCR-535: --- My mistakes. Sorry. It was caused by my first try to get rid of the bug. I will correct them ASAP. About the test case,

[jira] Updated: (JCR-535) Ignore root node when importing through sysView

2006-10-23 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-535?page=all ] Nicolas Toper updated JCR-535: -- Attachment: patch-WorkspaceImporter-231006-2.txt Here is the new patch. Ignore root node when importing through sysView

[jira] Commented: (JCR-535) Ignore root node when importing through sysView

2006-10-19 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-535?page=comments#action_12443580 ] Nicolas Toper commented on JCR-535: --- Understood :) Thanks. So I'll check the UUID (imagine a use case where we swap one JCR compliant repository for another; if I

[jira] Commented: (JCR-569) WorkspaceImporter Refactoring

2006-10-13 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-569?page=comments#action_12442223 ] Nicolas Toper commented on JCR-569: --- [[ Old comment, sent by email on Wed, 13 Sep 2006 18:00:23 +0200 ]] Hi Stefan, About your first point, you are right:

[jira] Commented: (JCR-580) Should be public: NodeType[] registerNodeTypes(List defs)

2006-09-25 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-580?page=comments#action_12437614 ] Nicolas Toper commented on JCR-580: --- Hi, Which class are you writing about please? Should be public: NodeType[] registerNodeTypes(List defs)

[jira] Updated: (JCR-569) WorkspaceImporter Refactoring

2006-09-17 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-569?page=all ] Nicolas Toper updated JCR-569: -- Attachment: WorkspaceImporter.patch Hi, As told, it was untested. Now it is and working (according to the unit test). BR Nicolas WorkspaceImporter Refactoring

[jira] Commented: (JCR-569) WorkspaceImporter Refactoring

2006-09-17 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-569?page=comments#action_12435345 ] Nicolas Toper commented on JCR-569: --- Just to be sure everything is all right: I have checked with my machine. Only 2 tests are run when calling

[jira] Commented: (JCR-569) WorkspaceImporter Refactoring

2006-09-13 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-569?page=comments#action_12434496 ] Nicolas Toper commented on JCR-569: --- Sure. Here it is: Introduction The current o.a.j.core.xml.WorspaceImporter class has one main responsability: to import data

[jira] Commented: (JCR-569) WorkspaceImporter Refactoring

2006-09-13 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-569?page=comments#action_12434497 ] Nicolas Toper commented on JCR-569: --- Hi Stefan, About your first point, you are right: this is why I sent yesterday a more in depth analysis. This JIRA issue will

[jira] Created: (JCR-569) WorkspaceImporter Refactoring

2006-09-12 Thread Nicolas Toper (JIRA)
WorkspaceImporter Refactoring - Key: JCR-569 URL: http://issues.apache.org/jira/browse/JCR-569 Project: Jackrabbit Issue Type: Improvement Reporter: Nicolas Toper Hi, As you know, I have run into

[jira] Updated: (JCR-569) WorkspaceImporter Refactoring

2006-09-12 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-569?page=all ] Nicolas Toper updated JCR-569: -- Attachment: SysViewImporter.patch This is the skeleton. This class is not to be included in JR code. WorkspaceImporter should inherit this class so no

[jira] Updated: (JCR-556) Refactoring of the BackupTool

2006-09-09 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-556?page=all ] Nicolas Toper updated JCR-556: -- Attachment: BIO.patch Add a flag autoCreate to know whether or not to create autocreate Node. As suggested by Jukka, it avoids to create a specific class to restore

[jira] Updated: (JCR-556) Refactoring of the BackupTool

2006-09-09 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-556?page=all ] Nicolas Toper updated JCR-556: -- Attachment: PropInfo.patch Set a flag in PropInfo in the apply method to escape or not the protected properties. This flag is by default set to the usual behavior

[jira] Updated: (JCR-556) Refactoring of the BackupTool

2006-09-04 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-556?page=all ] Nicolas Toper updated JCR-556: -- Attachment: patch-backup-040906.txt Hi, Here are some refactoring to the backup tool. This goes in the contrib package for the backup tool. It is mainly clean up of

[jira] Created: (JCR-556) Refactoring of the BackupTool

2006-09-01 Thread Nicolas Toper (JIRA)
Refactoring of the BackupTool - Key: JCR-556 URL: http://issues.apache.org/jira/browse/JCR-556 Project: Jackrabbit Issue Type: Improvement Reporter: Nicolas Toper Priority: Minor The

[jira] Updated: (JCR-556) Refactoring of the BackupTool

2006-09-01 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-556?page=all ] Nicolas Toper updated JCR-556: -- Attachment: patch-jr-010906-NodeVersionHistoriesUpdatableStateManager.txt This is a new class I propose to put in o.a.j.state. It is designed to update easily the

[jira] Updated: (JCR-556) Refactoring of the BackupTool

2006-09-01 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-556?page=all ] Nicolas Toper updated JCR-556: -- Attachment: patch-jr-010906-PropInfo.txt This patch contains a few updates I had to PropInfo. I added three getters and switched to other getXXX to public.

[jira] Updated: (JCR-556) Refactoring of the BackupTool

2006-09-01 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-556?page=all ] Nicolas Toper updated JCR-556: -- Attachment: patch-jr-010906-SysViewImporter.txt SysViewImporter: takes a system view XML document and import it as it is in an empty repository. It is heavily based

[jira] Updated: (JCR-556) Refactoring of the BackupTool

2006-09-01 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-556?page=all ] Nicolas Toper updated JCR-556: -- Attachment: patch-jr-010906-VersionManagerImpl.txt This patch adds importVersions in the VersionManagerImpl. Refactoring of the BackupTool

[jira] Updated: (JCR-556) Refactoring of the BackupTool

2006-09-01 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-556?page=all ] Nicolas Toper updated JCR-556: -- Attachment: patch-jr-010906-RepositoryImpl.txt This is a small update to RepositoryImpl. I wanted to avoid updating the interface VersionManager to add the method

[jira] Updated: (JCR-442) Implement a backup tool

2006-08-16 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=all ] Nicolas Toper updated JCR-442: -- Attachment: patch-jr-final1.tar.gz This is the proposed path to the core for the backup tool. Sorry for the 4 patches but some Gremlins have invaded my computer and

[jira] Created: (JCR-535) Ignore root node when importing through sysView

2006-08-08 Thread Nicolas Toper (JIRA)
Ignore root node when importing through sysView --- Key: JCR-535 URL: http://issues.apache.org/jira/browse/JCR-535 Project: Jackrabbit Issue Type: Improvement Components: core

[jira] Updated: (JCR-442) Implement a backup tool

2006-08-08 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=all ] Nicolas Toper updated JCR-442: -- Attachment: patch-backup-090806.txt Hopefully, this is one of the last patches for this project :) Everything is working but the importXML method in the

[jira] Updated: (JCR-442) Implement a backup tool

2006-08-07 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=all ] Nicolas Toper updated JCR-442: -- Attachment: patch-060808-backup.txt - A few classes has been cleaned up and are ready for production (Until BackupManager not included) - There is still the

[jira] Updated: (JCR-442) Implement a backup tool

2006-08-03 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=all ] Nicolas Toper updated JCR-442: -- Attachment: patch-jr-060803.txt Jackrabbit patch to core to allow restore. 2 updates: - Added to NamespaceRegistryImpl this method public boolean isRegistered(String

[jira] Created: (JCR-521) Add a method public boolean hasNodeType(String name) in NodeTypeManagerImpl

2006-07-28 Thread Nicolas Toper (JIRA)
Add a method public boolean hasNodeType(String name) in NodeTypeManagerImpl --- Key: JCR-521 URL: http://issues.apache.org/jira/browse/JCR-521 Project: Jackrabbit Issue

[jira] Updated: (JCR-521) Add a method public boolean hasNodeType(String name) in NodeTypeManagerImpl

2006-07-28 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-521?page=all ] Nicolas Toper updated JCR-521: -- Attachment: patch060728.txt Here is the patch implementing those changes. Add a method public boolean hasNodeType(String name) in NodeTypeManagerImpl

[jira] Updated: (JCR-521) Add a method public boolean hasNodeType(String name) in NodeTypeManagerImpl

2006-07-28 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-521?page=all ] Nicolas Toper updated JCR-521: -- Attachment: patch-060728-3.txt Correcting A and B Add a method public boolean hasNodeType(String name) in NodeTypeManagerImpl

[jira] Commented: (JCR-442) Implement a backup tool

2006-07-27 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=comments#action_12423945 ] Nicolas Toper commented on JCR-442: --- Hi, The first part of the project is completed!!! Backup is working just fine :) I am posting all patches ASAP. I am waiting

[jira] Updated: (JCR-442) Implement a backup tool

2006-07-27 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=all ] Nicolas Toper updated JCR-442: -- Attachment: patch-backup-060728.txt Working backup operation + relevant test case. No new classes added. A lot of issues remain: mostly documentation and code

[jira] Updated: (JCR-442) Implement a backup tool

2006-07-26 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=all ] Nicolas Toper updated JCR-442: -- Attachment: patch-backup-060726.txt Patch including (nearly) all corrections asked by Jukka. See comments please. Implement a backup tool ---

[jira] Updated: (JCR-442) Implement a backup tool

2006-07-26 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=all ] Nicolas Toper updated JCR-442: -- Attachment: patch-jackrabbit-060726.txt New patch. Implements corrections asked by Jukka. I changed a lot less methods Implement a backup tool

[jira] Commented: (JCR-442) Implement a backup tool

2006-07-26 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=comments#action_12423670 ] Nicolas Toper commented on JCR-442: --- Hi Jukka, Here are my answers :p a/ Done. Maybe I should put in the constructor of Backup the session since it's shared by a

[jira] Updated: (JCR-442) Implement a backup tool

2006-07-25 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=all ] Nicolas Toper updated JCR-442: -- Attachment: patch-backup-060725.txt first sizable patch. Not everything is working yet... - solved tab issue - renamed ManagerBackup as BackupManager - no more

[jira] Updated: (JCR-442) Implement a backup tool

2006-07-19 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=all ] Nicolas Toper updated JCR-442: -- Attachment: patch-backup-060719.txt Patch for the backup tool: 1/ Parsing of the configuration file is over 2/ Instanciation of the backup manager 3/ First draft of

[jira] Updated: (JCR-442) Implement a backup tool

2006-07-18 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=all ] Nicolas Toper updated JCR-442: -- Attachment: patch-jackrabbit-060718.txt a) Make RepositoryConfigurationParser a subclass of ConfigurationParser and use inheritance to access the protected methods

[jira] Updated: (JCR-442) Implement a backup tool

2006-07-15 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=all ] Nicolas Toper updated JCR-442: -- Attachment: patch-backup-060716.txt - XML configuration file is correctly parsed and interpreted by the Backup tool. - code compiles and is tested - Indentations is

[jira] Updated: (JCR-442) Implement a backup tool

2006-07-15 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=all ] Nicolas Toper updated JCR-442: -- Attachment: patch-jackrabbit-060716.txt - The new correct patch (I don't try to delete and recreate the same file within the same patch) - No tabs :p Implement a

[jira] Updated: (JCR-442) Implement a backup tool

2006-07-14 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=all ] Nicolas Toper updated JCR-442: -- Attachment: jackrabbit-1.patch.txt Jackrabbit patch with 2 classes for ConfigurationParser. I couldn't test the class since Jackrabbit does not currently compile

[jira] Updated: (JCR-442) Implement a backup tool

2006-07-11 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=all ] Nicolas Toper updated JCR-442: -- Attachment: patch.txt LaunchBackup tested and completed. (issue with Eclipse and SVN so please tell me if you have a problem importing the patch) Implement a

[jira] Updated: (JCR-442) Implement a backup tool

2006-07-05 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=all ] Nicolas Toper updated JCR-442: -- Attachment: patch.txt Deleting the backup package and restoring RepositoryImpl to its previous configuration. Implement a backup tool ---

[jira] Updated: (JCR-442) Implement a backup tool

2006-07-02 Thread Nicolas Toper (JIRA)
[ http://issues.apache.org/jira/browse/JCR-442?page=all ] Nicolas Toper updated JCR-442: -- Attachment: patch.txt ConfigBackup is now called BackupConfig. It is my first accepted patch on Apache Jackrabbit :) Implement a backup tool