fix issue #646

[JohnRDOrazio]

add lineIndex and lineString properties to a Property Node

issue #646

[JohnRDOrazio]

if this PR is accepted, perhaps documentation could be updated to explain how to access the properties of a node, and warn against simply trying to json_encode the resulting errors...

[JohnRDOrazio]

I added a Unit Test that will check for lineIndex and lineString properties when validating defective calendar data.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.78%. Comparing base (f14092f) to head (ec31757).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##             master     #649   +/-   ##
=========================================
  Coverage     98.78%   98.78%           
- Complexity     1866     1868    +2     
=========================================
  Files            71       71           
  Lines          5249     5253    +4     
=========================================
+ Hits           5185     5189    +4     
  Misses           64       64           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[phil-davis]

@JohnRDOrazio php-cs-fixer wants some formatting adjusted.
You can:
composer cs-fixer
and commit and push the changes it makes.

[phil-davis]

LGTM - this implements error handling features similar to what happens elsewhere (see the PR and issue #646 discussions.

[staabm]

I like the change!

[staabm]

thank you!