Mailing-List: contact dev-help@jackrabbit.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@jackrabbit.apache.org
Message-ID: <2123313647.1210163815705.JavaMail.jira@brutus>
Date: Wed, 7 May 2008 05:36:55 -0700 (PDT)
From: "Jukka Zitting (JIRA)" <jira@apache.org>
To: dev@jackrabbit.apache.org
Subject: [jira] Updated: (JCR-1574) [PATCH] PropertyWrapper has a typo that
 causes an infinite loop
In-Reply-To: <1487539882.1210122475762.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Jukka Zitting updated JCR-1574:
-------------------------------

    Fix Version/s:     (was: core 1.4.4)

AFAIK that method is never executed (PropertyWrapper is internal to the SAX event generator), so the issue is a bit theoretical.

Instead of fixing this as proposed, I'd rather get rid of the entire PropertyWrapper class (and the loads of unused code it contains) by refactoring the SAX event generation code.

> [PATCH] PropertyWrapper has a typo that causes an infinite loop
> ---------------------------------------------------------------
>
>                 Key: JCR-1574
>                 URL: https://issues.apache.org/jira/browse/JCR-1574
>             Project: Jackrabbit
>          Issue Type: Bug
>          Components: jackrabbit-core
>    Affects Versions: core 1.4.2
>            Reporter: Dave Brosius
>            Priority: Minor
>         Attachments: PropertyWrapper_typo_infinite_loop.patch
>
>
> PropertyWrapper.refresh is missing the delegation to 'property' for refresh, and has an infinite loop instead. Patch fixes this.
> [CODE] 
>    /** {@inheritDoc} */
>     public void refresh(boolean keepChanges) throws InvalidItemStateException,
>             RepositoryException {
>         refresh(keepChanges);
>     }
> [/CODE]

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