search

trs-80 / WWW-Spotify

Perl wrapper for Spotify Web API
Other
5 stars 3 forks source link

Use of XML::Simple is strongly discouraged #16

Closed trs-80 closed 2 years ago

trs-80 commented 2 years ago

PR tests were failing intermittently on pulling in XML::Simple (reason unknown) but when I reviewed that module to check on its status I discovered it is not recommended for use.

The plan would be to move to XML::LibXML::Simple which is designed to be close to a drop in replacement.

The XML calls have not been tested in the past so this will require changes to the test suite as well as changes to the dependencies.

We will also want to verify that the JSON and XML auto_decode methods return identical data structures, and if they do not then document the differences.

I consider this a low priority change as I am not aware of anyone using the XML format, if someone that is using the XML option reads this please let us know.

trs-80 commented 2 years ago

XML appears to be no longer supported, or at least there is no documentation available for it on the Spotify API site. Therefore I will shift the focus to removing all XML related items from a future release.

trs-80 commented 2 years ago

This was resolved in the release of 0.012 PR #17