eXide icon indicating copy to clipboard operation
eXide copied to clipboard

config:app-info() does not list dependencies in attributes

Open tgraham-antenna opened this issue 8 years ago • 1 comments

Moved from eXist-db/exist#1311:

What is the problem

The default index.html includes <div data-template="config:app-info"/>. That is supposed to generate "a table showing all properties defined in the application descriptors." However, <dependency> (and <permissions>) has attributes, and they don't show.

What did you expect

The dependency information.

This can be fixed by outputting all attributes (except password attributes):

Index: modules/config.xqm
===================================================================
--- modules/config.xqm	(revision 27934)
+++ modules/config.xqm	(working copy)
@@ -89,7 +89,7 @@
                 return
                     <tr>
                         <td>{node-name($attr)}:</td>
-                        <td>{$attr/string()}</td>
+                        <td>{$attr/string()}{string-join($attr/@* except $attr/@password, ' ')}</td>
                     </tr>
             }
             <tr>

Context information

  • eXist-db version + Git Revision hash : eXist-db 3.0 / acd0c14

tgraham-antenna avatar Mar 02 '17 17:03 tgraham-antenna

@tgraham-antenna Thanks for migrating the issue over here! Would you care to file a PR to the various locations that would benefit from your proposed change?

  • https://github.com/wolfgangmm/eXide/blob/develop/modules/config.xqm#L61-L83
  • https://github.com/wolfgangmm/eXide/blob/develop/templates/bootstrap/modules/config.xqm#L74-L100
  • https://github.com/wolfgangmm/eXide/blob/develop/templates/existdb/modules/config.xqm#L74-L100

joewiz avatar Mar 12 '17 21:03 joewiz