Prevent infinite loop on empty xml elements

[DeepDiver1975]

refs #132 and #146

About dropping code coverage: it is hard to make reader->read() to return false and hit these 5 lines.
The only possibility would be to mock the reader and fake this behavior - which is very unlikly.

I never the less would like to keep these checks since they can cause trouble if the occur.

ToDo

• backport to 1.5 - [1.5] Prevent infinite loop on empty xml elements #159
• backport to 2.0 - [2.0] Prevent infinite loop on empty xml elements #160
• backport to 2.1 - [2.1] Prevent infinite loop on empty xml elements #161
• release 2.2.0 with this fix
• release 2.1.2 - [2.1] Prevent infinite loop on empty xml elements #161
• release 2.0.1 - [2.0] Prevent infinite loop on empty xml elements #160
• release 1.5.1 - [1.5] Prevent infinite loop on empty xml elements #159

[codecov]

Codecov Report

Merging #158 into master will decrease coverage by 1.28%.
The diff coverage is 62.5%.

@@             Coverage Diff              @@
##             master     #158      +/-   ##
============================================
- Coverage     98.87%   97.58%   -1.29%     
  Complexity      112      112              
============================================
  Files            13       13              
  Lines           444      456      +12     
============================================
+ Hits            439      445       +6     
- Misses            5       11       +6

Impacted Files	Coverage Δ	Complexity Δ
lib/Deserializer/functions.php	89.69% <62.5%> (-5.61%)	0 <0> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fe7c953...cde4364. Read the comment docs.

[staabm]

after getting in the changes discussed in #159 we get move ahead.

thx

[rullzer]

👍 good stuff @DeepDiver1975