Item13235: DirectedGraphPlugin uses a case-sensitive test, possibly resulting in cruft attachments

Priority: Low
Current State: Closed
Released In: n/a
Target Release: n/a

Applies To: Extension
Component: DirectedGraphPlugin
Branches: master

Reported By: JohnKnutson
Waiting For:
Last Change By: GeorgeClark

At the end of commonTagsHandler is a test:

    if ( $3 && ( $3 eq 'dot' ) ) {
        _writeDebug("DirectedGraphPlugin - FOUND MATCH  -  $3");
        wrapupTagsHandler();
    }

That test is case sensitive. Given that the regex above that binds $3 is case-insensitive, the test should be

    if ( $3 && ( $3 =~ /^dot$/i ) ) {

-- JohnKnutson - 28 Jan 2015

Only affects documents where the close XML tag is not all lower-case, e.g. =</doT>

-- JohnKnutson - 28 Jan 2015

Summary	DirectedGraphPlugin uses a case-sensitive test, possibly resulting in cruft attachments
ReportedBy	JohnKnutson
Codebase	trunk
SVN Range
AppliesTo	Extension
Component	DirectedGraphPlugin
Priority	Low
CurrentState	Closed
WaitingFor
Checkins	DirectedGraphPlugin:871bf165959f
TargetRelease	n/a
ReleasedIn	n/a
CheckinsOnBranches	master
trunkCheckins
masterCheckins	DirectedGraphPlugin:871bf165959f
ItemBranchCheckins
Release01x01Checkins