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 2009/08/07 18:11:14 UTC

[jira] Created: (JCR-2249) Remove duplicate code in InternalValueFactory

Remove duplicate code in InternalValueFactory
---------------------------------------------

                 Key: JCR-2249
                 URL: https://issues.apache.org/jira/browse/JCR-2249
             Project: Jackrabbit Content Repository
          Issue Type: Improvement
          Components: jackrabbit-core
            Reporter: Marcel Reutegger
            Priority: Minor


After JCR-2245 has been applied some of the duplicate code in InternaValueFactory can be removed.

Namely:
- create(String, int)
- all other create methods that are non-binary ?

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


[jira] Updated: (JCR-2249) Remove duplicate code in InternalValueFactory

Posted by "Marcel Reutegger (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/JCR-2249?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Marcel Reutegger updated JCR-2249:
----------------------------------

    Attachment: JCR-2249.patch

Client code of InternalValueFactory depends on implmentation of returned QValue instances :-/

Updated patch. 

> Remove duplicate code in InternalValueFactory
> ---------------------------------------------
>
>                 Key: JCR-2249
>                 URL: https://issues.apache.org/jira/browse/JCR-2249
>             Project: Jackrabbit Content Repository
>          Issue Type: Improvement
>          Components: jackrabbit-core
>            Reporter: Marcel Reutegger
>            Priority: Minor
>         Attachments: JCR-2249.patch, JCR-2249.patch
>
>
> After JCR-2245 has been applied some of the duplicate code in InternaValueFactory can be removed.
> Namely:
> - create(String, int)
> - all other create methods that are non-binary ?

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


[jira] Updated: (JCR-2249) Remove duplicate code in InternalValueFactory

Posted by "Marcel Reutegger (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/JCR-2249?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Marcel Reutegger updated JCR-2249:
----------------------------------

    Attachment: JCR-2249.patch

Proposed changes.

> Remove duplicate code in InternalValueFactory
> ---------------------------------------------
>
>                 Key: JCR-2249
>                 URL: https://issues.apache.org/jira/browse/JCR-2249
>             Project: Jackrabbit Content Repository
>          Issue Type: Improvement
>          Components: jackrabbit-core
>            Reporter: Marcel Reutegger
>            Priority: Minor
>         Attachments: JCR-2249.patch
>
>
> After JCR-2245 has been applied some of the duplicate code in InternaValueFactory can be removed.
> Namely:
> - create(String, int)
> - all other create methods that are non-binary ?

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


[jira] Resolved: (JCR-2249) Remove duplicate code in InternalValueFactory

Posted by "Marcel Reutegger (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/JCR-2249?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Marcel Reutegger resolved JCR-2249.
-----------------------------------

       Resolution: Fixed
    Fix Version/s: 2.0.0

Applied latest patch in revision: 802755

> Remove duplicate code in InternalValueFactory
> ---------------------------------------------
>
>                 Key: JCR-2249
>                 URL: https://issues.apache.org/jira/browse/JCR-2249
>             Project: Jackrabbit Content Repository
>          Issue Type: Improvement
>          Components: jackrabbit-core
>            Reporter: Marcel Reutegger
>            Priority: Minor
>             Fix For: 2.0.0
>
>         Attachments: JCR-2249.patch, JCR-2249.patch
>
>
> After JCR-2245 has been applied some of the duplicate code in InternaValueFactory can be removed.
> Namely:
> - create(String, int)
> - all other create methods that are non-binary ?

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