Easy to break marc templates in the config file

[Mike Rylander]

If you have an XML comment in the <marctemplates> section of the opensrf.xml that controls open-ils.cat, you won't be able to create a new record. The problem is that the list delivered to the staff client and used to populate an internal hash contains '#comment' in the list, which can't be loaded, of course, because that's not a template, and also makes the hash unbalanced and breaks the backend data structure.

opensrf.settings should probably tell XML::Simple to ignore comments entirely, but short of that, it should should make sure to avoid including non-element child nodes in lists.

[Jason Stephenson (jstephenson)]

NOTE: This is a bug in OpenSRF that affects Evergreen.

While I confirmed this behavior in OpenSRF master just now, OpenSRF SettingsParser.pm uses XML::LibXML instead of XML::Simple.

The "problem" was in the XML2perl helper function in OpensRF::Utils::SettingsParser. It was ignoring nodes named "comment" but not also those named "#comment." (Who knew that XML::Libxml would add a # to the node name?)

I have pushed a branch that fixes it for me:

http://git.evergreen-ils.org/?p=working/OpenSRF.git;a=shortlog;h=refs/heads/user/dyrcona/lp1340982-ignore-settings-comments

This branch should be back ported to OpenSRF 2.5 if any more releases are planned.

[Galen Charlton (gmc)]

OpenSRF patches pushed to master, rel_3_0, and rel_2_5. Thanks, Jason!

[Galen Charlton (gmc)]

Setting the Evergreen side of it to fixed-released, as no supported version of Evergreen uses OpenSRF 2.5.x.