[Mondrian] //open/mondrian/src/main/mondrian/xmla/XmlaHandler.java Rev 41 and 42

Richard Emberson remberson at edgedynamics.com
Tue Jul 10 17:05:49 EDT 2007


In Rev 41 the writeHierarchyInfo method was changed as
follows by thiyagu:

private void writeHierarchyInfo(
                 SaxWriter writer,
                 Hierarchy[] hierarchies,
                 String[] props) {

             for (Hierarchy hierarchy : hierarchies) {
                 writer.startElement(
                     "HierarchyInfo", new String[]{
                     "name", hierarchy.getName()
                 });
                 for (final String prop : props) {
                     writer.element(
                         prop, getAttributes(prop, hierarchy));
                 }
                 writer.endElement(); // HierarchyInfo
             }
         }

         private String[] getAttributes(String prop, Hierarchy hierarchy) {
             String actualPropName = getPropertyName(prop);
             List<String> values = new ArrayList<String>();
             values.add("name");
             values.add(hierarchy.getUniqueName() + "." +
                     Util.quoteMdxIdentifier(actualPropName));
             if (longPropNames.get(prop) == null){
                 //Adding type attribute to the optional properties
                 values.add("type");
                 values.add(getXsdType(actualPropName));
             }
             return values.toArray(new String[values.size()]);
         }

Note that a "type" attribute could be included (the getAttributes
method was added).

With Rev 42 this was removed moving back to Rev 40 version by
jhyde.

The XmlaCognosTest.ref.xml file reflects Rev 41 not Rev 42 (or 40
and earlier).
Which is correct?


Richard
-- 
Quis custodiet ipsos custodes:
This email message is for the sole use of the intended recipient(s) and
may contain confidential information.  Any unauthorized review, use,
disclosure or distribution is prohibited.  If you are not the intended
recipient, please contact the sender by reply email and destroy all
copies of the original message.



More information about the Mondrian mailing list