You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jackrabbit.apache.org by "Marcel Reutegger (JIRA)" <ji...@apache.org> on 2008/04/16 12:25:22 UTC

[jira] Resolved: (JCR-1540) BundleDbPersistenceManager.checkConsistency() only fixes inconsistency if consistencyFix is enabled in configuration

     [ https://issues.apache.org/jira/browse/JCR-1540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Marcel Reutegger resolved JCR-1540.
-----------------------------------

       Resolution: Fixed
    Fix Version/s: 1.5

Fixed as proposed: 648660

> BundleDbPersistenceManager.checkConsistency() only fixes inconsistency if consistencyFix is enabled in configuration
> --------------------------------------------------------------------------------------------------------------------
>
>                 Key: JCR-1540
>                 URL: https://issues.apache.org/jira/browse/JCR-1540
>             Project: Jackrabbit
>          Issue Type: Bug
>          Components: jackrabbit-core
>    Affects Versions: 1.4, core 1.4.1, core 1.4.2
>            Reporter: Marcel Reutegger
>            Priority: Minor
>             Fix For: 1.5
>
>
> The method has a parameter that explicitly tells whether an inconsistency should be fixed, thus the configuration parameter should be ignored.
> Suggested patch:
> Index: BundleDbPersistenceManager.java
> ===================================================================
> --- BundleDbPersistenceManager.java	(revision 648657)
> +++ BundleDbPersistenceManager.java	(working copy)
> @@ -864,7 +864,7 @@
>          }
>  
>          // repair collected broken bundles
> -        if (consistencyFix && !modifications.isEmpty()) {
> +        if (fix && !modifications.isEmpty()) {
>              log.info(name + ": Fixing " + modifications.size() + " inconsistent bundle(s)...");
>              Iterator iterator = modifications.iterator();
>              while (iterator.hasNext()) {

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.