[jira] Created: (DIRSTUDIO-147) Allow LDIF imports to overwrite existing entries

Allow LDIF imports to overwrite existing entries
------------------------------------------------

                 Key: DIRSTUDIO-147
                 URL: https://issues.apache.org/jira/browse/DIRSTUDIO-147
             Project: Directory Studio
          Issue Type: Improvement
            Reporter: Paul Galbraith


Jarek Gawor's LDAP Browser/Editor has a valuable feature when doing LDIF imports:  the ability to import in "add", "update", or "update/add" mode.  This lets us create an LDIF file, import it, tweak it, and re-import it.  A simple checkbox in the LDIF import dialog to "update existing entries" would do the trick nicely (along with the supporting code to actually do the work!!).  

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

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

Stefan Seelmann updated DIRSTUDIO-147:
--------------------------------------

    Fix Version/s: 1.2.0
         Assignee: Stefan Seelmann

I wonder what would be the better strategy:

1) Try to add the entry, if we get an error 68 (Entry Already Exists) we do the update.
2) Check first via a search if the entry already exists, depending on the search result do an add or update.

> Allow LDIF imports to overwrite existing entries
> ------------------------------------------------
>
>                 Key: DIRSTUDIO-147
>                 URL: https://issues.apache.org/jira/browse/DIRSTUDIO-147
>             Project: Directory Studio
>          Issue Type: Improvement
>            Reporter: Paul Galbraith
>            Assignee: Stefan Seelmann
>             Fix For: 1.2.0
>
>
> Jarek Gawor's LDAP Browser/Editor has a valuable feature when doing LDIF imports:  the ability to import in "add", "update", or "update/add" mode.  This lets us create an LDIF file, import it, tweak it, and re-import it.  A simple checkbox in the LDIF import dialog to "update existing entries" would do the trick nicely (along with the supporting code to actually do the work!!).  

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

    [ https://issues.apache.org/jira/browse/DIRSTUDIO-147?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12564763#action_12564763 ]

Paul Galbraith commented on DIRSTUDIO-147:
------------------------------------------

I'd vote for option 1, it seems to imply fewer remote calls, but I'm no ldap expert...

> Allow LDIF imports to overwrite existing entries
> ------------------------------------------------
>
>                 Key: DIRSTUDIO-147
>                 URL: https://issues.apache.org/jira/browse/DIRSTUDIO-147
>             Project: Directory Studio
>          Issue Type: Improvement
>            Reporter: Paul Galbraith
>            Assignee: Stefan Seelmann
>             Fix For: 1.2.0
>
>
> Jarek Gawor's LDAP Browser/Editor has a valuable feature when doing LDIF imports:  the ability to import in "add", "update", or "update/add" mode.  This lets us create an LDIF file, import it, tweak it, and re-import it.  A simple checkbox in the LDIF import dialog to "update existing entries" would do the trick nicely (along with the supporting code to actually do the work!!).  

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

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

Pierre-Arnaud Marcelot updated DIRSTUDIO-147:
---------------------------------------------

    Component/s: studio-ldapbrowser

> Allow LDIF imports to overwrite existing entries
> ------------------------------------------------
>
>                 Key: DIRSTUDIO-147
>                 URL: https://issues.apache.org/jira/browse/DIRSTUDIO-147
>             Project: Directory Studio
>          Issue Type: Improvement
>          Components: studio-ldapbrowser
>            Reporter: Paul Galbraith
>            Assignee: Stefan Seelmann
>             Fix For: 1.2.0
>
>
> Jarek Gawor's LDAP Browser/Editor has a valuable feature when doing LDIF imports:  the ability to import in "add", "update", or "update/add" mode.  This lets us create an LDIF file, import it, tweak it, and re-import it.  A simple checkbox in the LDIF import dialog to "update existing entries" would do the trick nicely (along with the supporting code to actually do the work!!).  

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

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

Pierre-Arnaud Marcelot updated DIRSTUDIO-147:
---------------------------------------------

    Fix Version/s:     (was: 1.2.0)
                   1.3.0

Postponed.

> Allow LDIF imports to overwrite existing entries
> ------------------------------------------------
>
>                 Key: DIRSTUDIO-147
>                 URL: https://issues.apache.org/jira/browse/DIRSTUDIO-147
>             Project: Directory Studio
>          Issue Type: Improvement
>          Components: studio-ldapbrowser
>            Reporter: Paul Galbraith
>            Assignee: Stefan Seelmann
>             Fix For: 1.3.0
>
>
> Jarek Gawor's LDAP Browser/Editor has a valuable feature when doing LDIF imports:  the ability to import in "add", "update", or "update/add" mode.  This lets us create an LDIF file, import it, tweak it, and re-import it.  A simple checkbox in the LDIF import dialog to "update existing entries" would do the trick nicely (along with the supporting code to actually do the work!!).  

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

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

Stefan Seelmann resolved DIRSTUDIO-147.
---------------------------------------

    Resolution: Fixed

Fixed. It is implemented using REPLACE operations for each attribute in the LDIF. Note that attributes of the existing entry that are not defined in the LDIF record remain without modification. For example if the entry has an attribute 'mail' but the LDIF doesn't contain such an attribute then the 'mail' attribute remains in the entry. There is no delete operation.

Fixed with following commits:
  http://svn.apache.org/viewvc?rev=689285&view=rev
  http://svn.apache.org/viewvc?rev=689388&view=rev

> Allow LDIF imports to overwrite existing entries
> ------------------------------------------------
>
>                 Key: DIRSTUDIO-147
>                 URL: https://issues.apache.org/jira/browse/DIRSTUDIO-147
>             Project: Directory Studio
>          Issue Type: Improvement
>          Components: studio-ldapbrowser
>            Reporter: Paul Galbraith
>            Assignee: Stefan Seelmann
>             Fix For: 1.3.0
>
>
> Jarek Gawor's LDAP Browser/Editor has a valuable feature when doing LDIF imports:  the ability to import in "add", "update", or "update/add" mode.  This lets us create an LDIF file, import it, tweak it, and re-import it.  A simple checkbox in the LDIF import dialog to "update existing entries" would do the trick nicely (along with the supporting code to actually do the work!!).  

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