? ? ? Pending

User tests: Successful: Unsuccessful:

avatar imanickam
imanickam
15 Nov 2018

Current:
http://www.php.net/manual/en/function.date.php - this URL does not work; it requires signing into the php.net site

Proposed:
http://php.net/date - this URL works and is consistent with other language strings MOD_ARTICLES_CATEGORY_FIELD_DATEFIELDFORMAT_DESC and MOD_ARTICLES_CATEGORY_FIELD_MONTHYEARFORMAT_DESC

Pull Request for Issue # .

Summary of Changes

Current language string:
JGLOBAL_DATE_FORMAT_DESC="Optional format string for showing the date. For example, D M Y for day month year or you can use d-m-y for a short version eg. 28-12-16. See http://www.php.net/manual/en/function.date.php. If left blank, it uses DATE_FORMAT_LC1 from your language file."

Revised language string:
JGLOBAL_DATE_FORMAT_DESC="Optional format string for showing the date. For example, D M Y for day month year or you can use d-m-y for a short version eg. 28-12-16. See http://php.net/date. If left blank, it uses DATE_FORMAT_LC1 from your language file."

Testing Instructions

(a) Go to the screen "Articles: Options" (Content => Articles > Options)
(b) Click on the tab List Layouts
(c) For the option "Show Date", choose the value Created or Modified or Published)
(d) Move the cursor over the label "Date Format" - Review the pop-up help text
(e) Type the URL shown in the pop-up help text in a separate window/tab of a browser and see what happens

Expected result

(a) Pop-up Help Text

after_language_string_change

(b) URL to the PHP Date Function taken from the pop-up Help Text

php_function-date_url_without_www

Actual result

(a) Pop-up Help Text

before_language_string_change

(b) URL to the PHP Date Function taken from the pop-up Help Text

php_function-date_url_with_www

Documentation Changes Required

avatar imanickam imanickam - open - 15 Nov 2018
avatar imanickam imanickam - change - 15 Nov 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Nov 2018
Category Administration Language & Strings
avatar PhilETaylor
PhilETaylor - comment - 15 Nov 2018

http://php.net/manual/en/function.date.php works fine for me here without authentication. Im guessing you are seeing a mirror site once bounced? Personally I like the shorter url that you propose as its normally quicker to type ;-)

HTTP/1.1 200 OK
Connection: keep-alive
Content-Encoding: gzip
Content-Type: text/html; charset=utf-8
Content-language: en
Date: Thu, 15 Nov 2018 06:20:59 GMT
Last-Modified: Thu, 15 Nov 2018 05:04:46 GMT
Link: <http://php.net/date>; rel=shorturl
Server: nginx/1.10.3
Set-Cookie: LAST_LANG=en; expires=Fri, 15-Nov-2019 06:20:59 GMT; Max-Age=31536000; path=/; domain=.php.net
Set-Cookie: COUNTRY=NA%2C87.244.71.75; expires=Thu, 22-Nov-2018 06:20:59 GMT; Max-Age=604800; path=/; domain=.php.net
Transfer-Encoding: chunked
Vary: Accept-Encoding
X-Frame-Options: SAMEORIGIN
avatar infograf768
infograf768 - comment - 15 Nov 2018

I have tested this item successfully on 6cbe0bc

If this may help some users, let's change it.


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

avatar infograf768 infograf768 - test_item - 15 Nov 2018 - Tested successfully
avatar PhilETaylor
PhilETaylor - comment - 15 Nov 2018

yes I agree, I think the redirect also handles the language param in the url right as well - another reason to change it.

avatar PhilETaylor
PhilETaylor - comment - 15 Nov 2018

Just being picky but can we use https://secure.php.net/date instead of http://php.net/date ?

avatar PhilETaylor
PhilETaylor - comment - 15 Nov 2018
avatar PhilETaylor
PhilETaylor - comment - 15 Nov 2018

I have tested this item successfully on 6cbe0bc


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

avatar PhilETaylor PhilETaylor - test_item - 15 Nov 2018 - Tested successfully
avatar infograf768 infograf768 - change - 15 Nov 2018
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 15 Nov 2018

RTC


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

avatar brianteeman
brianteeman - comment - 15 Nov 2018

This is clearly an upstream problem with the mirror that you are being directed to. Have you reported it upstream so it can be fixed for everyone that uses php

avatar imanickam
imanickam - comment - 15 Nov 2018

This is clearly an upstream problem with the mirror that you are being directed to. Have you reported it upstream so it can be fixed for everyone that uses php

Yes, the problem has been reported - Bug #77161 - PHP.net Website Problem (https://bugs.php.net/bug.php?id=77161).

avatar PhilETaylor
PhilETaylor - comment - 15 Nov 2018

Might be worth telling the PHP guys where you are in the world, and maybe your IP because then they might be able to "guess" which mirror you get redirected to.

avatar imanickam
imanickam - comment - 15 Nov 2018

Might be worth telling the PHP guys where you are in the world, and maybe your IP because then they might be able to "guess" which mirror you get redirected to.

Good Idea. I thought that the IP address would be stored along with the bug report when you create one, just like we can find out from which IP address the post has been created in the Joomla! Forum.

avatar imanickam imanickam - change - 15 Nov 2018
Labels Added: ?
avatar imanickam imanickam - change - 15 Nov 2018
Labels Added: ? ?
avatar imanickam
imanickam - comment - 15 Nov 2018

Standardized the URL that references the date function in PHP from http://php.net/date to https://php.net/date.

This has been changed in the file en-GB.mod_articles_category.ini (that is located in the directory \language\en-GB) as well.

The URLs are changed in the language strings MOD_ARTICLES_CATEGORY_FIELD_DATEFIELDFORMAT_DESC and MOD_ARTICLES_CATEGORY_FIELD_MONTHYEARFORMAT_DESC

avatar mbabker mbabker - change - 17 Nov 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-11-17 17:24:48
Closed_By mbabker
avatar mbabker mbabker - close - 17 Nov 2018
avatar mbabker mbabker - merge - 17 Nov 2018

Add a Comment

Login with GitHub to post a comment