GNOME Bugzilla – Bug 331273
Refactor usage of xmlIsID() in tree.c
Last modified: 2021-07-05 13:26:37 UTC
In tree.c xmlIsID() is used superfluously in some cases. This could have unnecessary negative impact on copy-node operations if DTD are used. Additionally, due to XInclude, in xmlCopyPropInternal() attributes will stay marked as ID, even if the target document does not define them as IDs; except for xml:id and attributes set to ID by an API, this is not correct. The issue gets more complex due to XInclude, of whose semantics semantics/problems I don't have a picture. Relevant mail-thread: http://mail.gnome.org/archives/xml/2006-February/msg00011.html Note that I started implementing the function xmlDOMWrapClodeNode() in tree.c to handle such issues for DOM-wrappers.
GNOME is going to shut down bugzilla.gnome.org in favor of gitlab.gnome.org. As part of that, we are mass-closing older open tickets in bugzilla.gnome.org which have not seen updates for a longer time (resources are unfortunately quite limited so not every ticket can get handled). If you can still reproduce the situation described in this ticket in a recent and supported software version, then please follow https://wiki.gnome.org/GettingInTouch/BugReportingGuidelines and create a new ticket at https://gitlab.gnome.org/GNOME/libxml2/-/issues/ Thank you for your understanding and your help.