Update Node.php

[Julyanvdham]

This edit makes sure that backslashes are not removed when the chart is made.

[jaapio]

Could you please provide a testcase that covers this behavior?

[ashnazg]

NodeTest::testToStringWithLabelContainingSlashes() says that this current behavior is expected:

\\\\phpDocumentor\\\\Descriptor\\\\ProjectDescriptor

After this patch, the result becomes:

\\phpDocumentor\\Descriptor\\ProjectDescriptor

@mvriel @jaapio , is the existing test incorrect in its expected result?

[ashnazg]

Judging from cc37321, this backslashing is intentional.

[jaapio]

I don't know if it is intentional. @mvriel could know the answer but it seems to be long time ago that this was introduced. Since this PR doesn't change any tests and does break tests if seems to be invalid to me. Tests are leading for me when in doubt.

[ashnazg]

See AttributeTest::testToStringWithSpecials()... Double backslashes should stay the same is one of the assertion messages. I'm fairly convinced by now that the existing behavior is intentional and correct.