Opened 5 years ago

Closed 5 years ago

#2626 closed bug (fixed)

make Format.ArcXML tests pass in Safari

Reported by: bartvde Owned by:
Priority: minor Milestone: 2.10 Release
Component: Format.ArcXML Version: 2.9
Keywords: Cc:
State: Complete

Description

Parse errors are dealt with differently in Safari then the code now assumes.

Attachments (1)

ticket2626.patch (471 bytes) - added by bartvde 5 years ago.

Download all attachments as: .zip

Change History (4)

Changed 5 years ago by bartvde

comment:1 Changed 5 years ago by bartvde

  • State set to Review

This one-line patch fixes up the test in Safari for ArcXML and ArcXML/Features, please review

comment:2 Changed 5 years ago by elemoine

  • State changed from Review to Commit

Bart, could you please add a comment in the test to quickly explain why this extra check (arcNode.firstChild.nodeName === 'parsererror') is needed? Thanks.

comment:3 Changed 5 years ago by bartvde

  • Resolution set to fixed
  • State changed from Commit to Complete
  • Status changed from new to closed

(In [10302]) make Format.ArcXML tests pass in Safari, invalid XML documents are dealt with differently in Safari, r=elemoine (closes #2626)

Note: See TracTickets for help on using tickets.