User tests: Successful: Unsuccessful:
PR #28549 added a force ltr character before the version number is displayed to prevent issues when using an rtl language.
It should not have been added to this instance as it is only used for the update log file which is always in english and therefore LTR and the special character is not recognised in plain text
code review as its an obvious fix
the logs\joomla_update.php file will contain a line similar to
2021-12-30T08:12:30+00:00 INFO ::1 update Update started by user Brian Teeman (608). Old version is ‎4.0.6-dev.
the logs\joomla_update.php file will contain a line similar to
2021-12-30T08:12:30+00:00 INFO ::1 update Update started by user Brian Teeman (608). Old version is 4.0.6-dev.
none
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_joomlaupdate |
I have tested this item
Code review.
@brianteeman Regarding the RTL display of version numbers at other places than the log (for which I had started to work on the 3.10-dev branch as you maybe remember): If there are some issues remaining after your CSS cleanups, I can make a PR to fix them. I have already prepared most of it. In opposite to my previous attempt I will not use a helper but it will work the same way, wrap it into a span with dir=ltr if necessary. But I think it has been wrong for so long time and we had not many issues with it, so I think there is no need to hurry. Let me know if you disagree.
It should be plain text with extension "log" or "txt"
it’s a PHP file for security - when accessed through a direct url it doesn’t render the content but dies. (Or should do)
a txt or log file in the webspace would be publicly available to view which could be bad.
It should be plain text with extension "log" or "txt"
it’s a PHP file for security - when accessed through a direct url it doesn’t render the content but dies. (Or should do)
a txt or log file in the webspace would be publicly available to view which could be bad.
@PhilETaylor I know. But that could be solved differently or the PHP log could use HTML markup or something like that.
changing the file type is beyond the scope of this PR
changing the file type is beyond the scope of this PR
@brianteeman Yes, I know, that's why I have tested the PR with success.
I have tested this item
Status | Pending | ⇒ | Ready to Commit |
RTC
RTC
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-12-30 12:37:12 |
Closed_By | ⇒ | wilsonge | |
Labels |
Added:
?
?
|
Thanks!
I agree that no HTML markup (and the unicode LTR mark can be considered as such) shall appear in a plain text log file.
Unfortunately the log is a PHP file, so some people might have some extension which shows it in the backend and so in a browser. Maybe this is how it came that the LTR mark was added.
I think it was wrong from the beginning on to use a PHP file as log file. It should be plain text with extension "log" or "txt".
If using PHP, the content should me marked somehow as text only, or it should produce HTML markup (with which we then could use the LTR marks or better ways again to show the version with LTR direction in the right way).
We should consider to change that for 5.0.