Feeling Lucky
?
?
[#34452] - [4.0] Replace double quote with single quote
- Fixed in Code Base
- 9 Jun 2021
- Medium
- Build: 4.0-dev
- # 34452
- Diff
- Quy:single-quote
User tests: Successful: Unsuccessful:
Summary of Changes
For consistency, replace double quote with singe quote.
Testing Instructions
Code review.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_associations com_redirect Front End com_content Layout Libraries Plugins |
Bakual
- comment
- 7 Jun 2021
what it is supposed to be standard? if there is 1
There is no official standard.
HLeithner
- comment
- 7 Jun 2021
Always use single quote except you want to use fast rope which is faster but personally I don't variables within a text block, it's less readable without a syntax highlighter.
alikon
- comment
- 8 Jun 2021
I have tested this item
code review
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34452.
joomdonation
- comment
- 9 Jun 2021
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34452.
| Status | Pending | ⇒ | Ready to Commit |
joomdonation
- comment
- 9 Jun 2021
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34452.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-06-09 07:17:02 |
| Closed_By | ⇒ | drmenzelit | |
| Labels |
Added:
?
?
|
||
drmenzelit
- comment
- 9 Jun 2021
Thanks
what it is supposed to be standard? if there is 1