Issue 110927 - configmgr's getHierarchicalName is broken
Summary: configmgr's getHierarchicalName is broken
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: scripting (show other issues)
Version: DEV300m76
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo 3.3
Assignee: Frank Schönheit
QA Contact: issues@framework
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2010-04-16 07:10 UTC by Frank Schönheit
Modified: 2017-05-20 10:24 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments
Basic macro to illustrate the problem (720 bytes, text/plain)
2010-04-16 07:12 UTC, Frank Schönheit
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description Frank Schönheit 2010-04-16 07:10:55 UTC
In some recent builds, before the great configmgr renewal, the following patter
used to work:
- create a configuration node (at the config provider)
- retrieve one of its sub nodes
- ask the sub node for its hierarchical name
  XHierarchicalName::getHierarchicalName
- create a new configuration node (at the config provider) using this
  hierarchical name

In m76, this doesn't work anymore. The reason is that getHierarchicalName now
returns a string which is no valid configuration node path anymore.

See the attached Basic macro to illustrate this: It displays the local and the
fully qualified name of a config sub node (getName and getHierarchicalName). In
m73 and earlier, the latter is a fully qualified node name, in m74 and later, it
isn't anymore.

Not sure if getByHierarchicalName was ever intended to provide a "re-usable"
node path,
http://api.openoffice.org/docs/common/ref/com/sun/star/configuration/HierarchyElement.html#XHierarchicalName
is not very clear about this. However, I don't see another way to obtain such a
complete node path from an existing node, so getByHierarchicalName seems the
canonic choice for this purpose.
Comment 1 Frank Schönheit 2010-04-16 07:12:49 UTC
Created attachment 68955 [details]
Basic macro to illustrate the problem
Comment 2 Stephan Bergmann 2010-04-16 10:44:27 UTC
fixed as <http://hg.services.openoffice.org/cws/slidecopy/rev/d88273e50dfe>
Comment 3 Stephan Bergmann 2010-04-16 10:50:21 UTC
.
Comment 4 Stephan Bergmann 2010-04-16 10:50:41 UTC
.
Comment 5 Frank Schönheit 2010-04-29 08:16:38 UTC
works fine in CWS slidecopy now => VERIFIED