[Bug 26226] msxml4 emits too much xmlns attributes

[wine-bugs at winehq.org]

http://bugs.winehq.org/show_bug.cgi?id=26226

--- Comment #32 from Ulrik Dickow <u.dickow at gmail.com> 2012-06-11 19:04:56 CDT ---
Created attachment 40511
  --> http://bugs.winehq.org/attachment.cgi?id=40511
msxml3: Support xmlns[:*] attribute nodes intelligently.

This patch, number 2 of 2, is where the new code comes in.  The 2 patches here
are based on the latest Wine git master as of this moment (updated 4 hours ago
according to gitweb).  The "1.5.6-fix-beta4" mentioned in the earlier
attachments is the pure 1.5.6 with the same patches applied, tested earlier on
June 11, before anything newer than 1.5.6 was visible on gitweb (Sourceforge). 
Fortunately the official updates after 1.5.6 didn't touch the same files as my
patches, so the tests should still be valid.

About 4 hours ago I tested on the Wine testbot that 'msxml3_crosstest.exe
domdoc' didn't produce any errors with this latest version of my 2 patches
combined, appplied to pure 1.5.6.  (The testbot was helpful in revealing bugs
in my beta2 version earlier in the day).

The patch doesn't produce "optimal" XML, i.e. with a redundant xmlns suppressed
in children as seen from a parent, but it now produces fully valid XML, with
successful return code (S_OK), in many more cases than before.  It makes the
BridgeCentral SOAP login work (i.e. that server doesn't care about extra
whitespace and repeated xmlns:SOAP-ENV and xmlns in children, as long as they
aren't any more doubled in the same element/level).

I hope the patch will also make the financial app that originally triggered
this bug report work.

-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
Do not reply to this email, post in Bugzilla using the
above URL to reply.
------- You are receiving this mail because: -------
You are watching all bug changes.