Comment 10 for bug 812593

Revision history for this message
Lebbeous Fogle-Weekley (lebbeous) wrote :

Here's a branch I'd like somebody to test that might deal with some of the problems listed in this ticket.

Any problems with ? : ' + [ ] in titles and other lineitem fields should be fixed by this (I think from that set there were only really problems with ? and [ ], and for ? only when ? was the last character in a string).

Any problems with characters with diacritics (and really any non ASCII characters) in lineitem fields should be fixed by this.

Regarding [ and ], I could not actually reproduce a problem with those characters in a title or other lineitem attribute field. There's no reason not to go ahead and strip them out though, since the fields we're affecting in the EDI output are really there only for visual fallback for the vendor, and aren't used by machines to look anything up, as I understand it.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/collab/senator/edi-translation-improvements

Feedback desired. Thanks!