? Success

User tests: Successful: Unsuccessful:

avatar smanzi
smanzi
23 Nov 2014

Just a tiny cosmetic fix:

I do not know if this is wanted for political or legal reasons, but in Extensions Manager, Editor - None is listed as having an "Unknown" author.
I fixed this by adding the required info in none.xml
There is an inconsistency, anyway: the "Creation Date" is set as "August 2004", while the standard Copyright notice says "2005 -2014"

avatar smanzi smanzi - open - 23 Nov 2014
avatar jissues-bot jissues-bot - change - 23 Nov 2014
Labels Added: ?
avatar smanzi
smanzi - comment - 23 Nov 2014

I think this untestable with com_patchtester, but there shouldn't be any possibility of bug, I guess.
If there is no legal objection I think this can be merged...

avatar zero-24
zero-24 - comment - 23 Nov 2014

@smanzi good find. I think there is a theoretic way to test :D

  • see plugin editor none
  • apply patch
  • refresh manifest data
  • see plugin editor none

Bevor the patch
screen shot 2014-11-23 at 03 54 40

After the patch
screen shot 2014-11-23 at 03 54 55

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

avatar zero-24 zero-24 - test_item - 23 Nov 2014 - Tested successfully
avatar zero-24
zero-24 - comment - 23 Nov 2014

There is an inconsistency, anyway: the "Creation Date" is set as "August 2004", while the standard Copyright notice says "2005 -2014"

Confirmed. I think the comitter can have a look into it.

avatar chrisdavenport
chrisdavenport - comment - 23 Nov 2014

It probably originated with Mambo.

avatar zero-24
zero-24 - comment - 23 Nov 2014

@chrisdavenport so the author tag needs to go to Mambo?

avatar chrisdavenport
chrisdavenport - comment - 23 Nov 2014

No, I think it's probably safe to keep the author as Joomla. The code will have changed because Joomla uses plugins whereas in Mambo it would have been a "mambot". I was just speculating on why the creation date predates Joomla. A better fix might be to alter the creation date to when Joomla 1.0.0 was released.

avatar zero-24
zero-24 - comment - 23 Nov 2014

A better fix might be to alter the creation date to when Joomla 1.0.0 was released.

So we should use September 2005 instead of August 2004

avatar chrisdavenport
chrisdavenport - comment - 23 Nov 2014

Yes.

avatar zero-24
zero-24 - comment - 23 Nov 2014

thanks @chrisdavenport
So @smanzi can you add this in to your PR?

avatar smanzi
smanzi - comment - 23 Nov 2014

Fixed!
@zero-24 Oh yes, silly me, it is just a matter of "Refresh Cache"! I previously tried with "Discover", but this had no effect....

avatar Bakual Bakual - reference | 15f157c - 23 Nov 14
avatar Bakual Bakual - merge - 23 Nov 2014
avatar Bakual Bakual - close - 23 Nov 2014
avatar Bakual Bakual - close - 23 Nov 2014
avatar Bakual Bakual - change - 23 Nov 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-11-23 13:08:51
avatar Bakual
Bakual - comment - 23 Nov 2014

Good catch. Thanks!

avatar Bakual Bakual - change - 23 Nov 2014
Milestone Added:
avatar smanzi smanzi - head_ref_deleted - 23 Nov 2014

Add a Comment

Login with GitHub to post a comment