git.net

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[GitHub] ant-ivy issue #73: Enable XML report parser to produce qualified extra attri...


Github user aprelev commented on the issue:

    https://github.com/apache/ant-ivy/pull/73
  
    > Rather than hardcoding prefix, shouldn't we retrieve it by URL "http://ant.apache.org/ivy/extra";?
    
    Can you elaborate on prefix retrieving? The **extra-** prefix is [hardcoded at](https://github.com/apache/ant-ivy/blob/5918182e0d6836d89c42260da9de4428d4cbcec0/src/java/org/apache/ivy/plugins/report/XmlReportWriter.java#L175) `XmlReportWriter::extraToString()` regardless of attribute namespace:
    ```Java
    sb.append(prefix).append("extra-").append(entry.getKey()).append("=\"")
        .append(XMLHelper.escape(entry.getValue())).append("\"");
    ```
    
    Maybe including namespace in XML report by passing *qualified* attributes to `XmlReportWriter::extraToString()` would be a better choice then?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@xxxxxxxxxxxxxx
For additional commands, e-mail: dev-help@xxxxxxxxxxxxxx




( ! ) Warning: include(msgfooter.php): failed to open stream: No such file or directory in /var/www/git/apache-ant-development/msg02293.html on line 90
Call Stack
#TimeMemoryFunctionLocation
10.0007358376{main}( ).../msg02293.html:0

( ! ) Warning: include(): Failed opening 'msgfooter.php' for inclusion (include_path='.:/var/www/git') in /var/www/git/apache-ant-development/msg02293.html on line 90
Call Stack
#TimeMemoryFunctionLocation
10.0007358376{main}( ).../msg02293.html:0