? ? Pending

User tests: Successful: Unsuccessful:

avatar Fedik
Fedik
8 Sep 2022

Summary of Changes

Tried to make a better explanation of using $jsSafe parameter in Text::script() and Text::_()

Testing Instructions

Text review

avatar Fedik Fedik - open - 8 Sep 2022
avatar Fedik Fedik - change - 8 Sep 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Sep 2022
Category Libraries
avatar Fedik
Fedik - comment - 8 Sep 2022

@nikosdion here is PR about jsSafe doc block

avatar nikosdion
nikosdion - comment - 8 Sep 2022

I like it :) It is clear and to the point.

avatar Fedik Fedik - change - 8 Sep 2022
Labels Added: ?
avatar brianteeman
brianteeman - comment - 8 Sep 2022

I have tested this item successfully on a3ce624


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38726.

avatar brianteeman brianteeman - test_item - 8 Sep 2022 - Tested successfully
avatar toivo
toivo - comment - 9 Sep 2022

I have tested this item successfully on a3ce624


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38726.

avatar toivo toivo - test_item - 9 Sep 2022 - Tested successfully
avatar richard67 richard67 - change - 9 Sep 2022
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 9 Sep 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38726.

avatar laoneo laoneo - change - 12 Sep 2022
Labels Added: ?
avatar fancyFranci fancyFranci - close - 14 Sep 2022
avatar fancyFranci fancyFranci - merge - 14 Sep 2022
avatar fancyFranci fancyFranci - change - 14 Sep 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-09-14 14:12:56
Closed_By fancyFranci
avatar fancyFranci
fancyFranci - comment - 14 Sep 2022

Indeed clearer. Thank you!

Add a Comment

Login with GitHub to post a comment