PhocaCz / PhocaDownload

Phoca Download is Joomla! CMS extension. Component which displays downloadable documents on website.
http://www.phoca.cz/phocadownload
Other
10 stars 6 forks source link

Remove deprecated function JString #2

Closed photodude closed 7 years ago

photodude commented 8 years ago

JString has been depreciated use Joomla\String\StringHelper;

This PR updates the component to use the latest String package due to a B/C break in the Joomla CMS 3.5+ supporting one of the changes in PHP 7 resulting in new reserved keywords in the language.

Reference on this change requirement from the Joomla CMS https://github.com/joomla/joomla-cms/pull/6600

PhocaCz commented 8 years ago

Hi, thank you for this info.

Is it sure, this functions will stay in Joomla!?

libraries\vendor\joomla\string\src\StringHelper.php

Are there any info, what the vendor means?

Thank you very much.

Jan

Dne 14. 12. 2015 v 20:08 photodude napsal(a):

JString has been depreciated |use Joomla\String\StringHelper;|

This PR updates the component to use the latest String package due to a B/C break in the Joomla CMS 3.5+ supporting one of the changes in PHP 7 resulting in new reserved keywords in the language.

Reference on this change requirement from the Joomla CMS joomla/joomla-cms#6600 https://github.com/joomla/joomla-cms/pull/6600


    You can view, comment on, or merge this pull request online at:

https://github.com/PhocaCz/PhocaDownload/pull/2

    Commit Summary

— Reply to this email directly or view it on GitHub https://github.com/PhocaCz/PhocaDownload/pull/2.

photodude commented 8 years ago

Vendor is just referencing "external libraries" to the Joomla CMS; since the string library is part of the Joomla Framework it's included as a "vendor library" rather than as a library of the CMS. (I know a bit confusing)

Long story short, it is a Joomla library and it will stay in the Joomla CMS as the official string helper library from now on. It's just a name and location change from the old Joomla platform JString → to the namespaced Joomla Framework String library → to the PHP7 compatible namespaced Joomla Framework StringHelper library

All the best, Walt