@halfdan opened this Issue on January 26th 2014 Member

The plugin listing (Platform > Plugins) currently uses undocumented keys from plugin.json to show the author and links to the author's homepage.

Authors can be added in plugin.json with an "authors" array (e.g. https://github.com/piwik/plugin-SecurityInfo/blob/master/plugin.json). However the plugin listing uses "author" and "author_homepage".

I think it is reasonable to simply show all the authors in a row (e.g. By Foo, Bar, Baz) with link to their homepage.

@mattab commented on January 27th 2014 Owner

Thanks for the report!

Indeed author_homepage is found many times in the source code, but not documented at: http://developer.piwik.org/guides/distributing-your-plugin

@tsteur, what are your thoughts on this one?

@halfdan commented on January 27th 2014 Member

IMHO author and author_homepage should be removed in favor of the new "authors" array.

FYI, I've set myself as owner since I have the fix ready :P

@tsteur commented on January 27th 2014 Owner

+1 for removing author_homepage

@anonymous-piwik-user commented on January 28th 2014

In 4fcd238e8118f94a5fa86248821882a699a52817: Show list of plugin authors with homepage and use actual data from plugin.json

refs #4578

@anonymous-piwik-user commented on January 28th 2014

In cae15a35646c1ca5581462ddea7a62da04d407d2: Convert the old author/author_homepage to the new structure.

refs #4578

@halfdan commented on January 28th 2014 Member

So - I converted all the old occurences in Piwik to the new data structure. This got me thinking: Do we need to show "By Piwik" on every core plugin? It's kinda obvious that the core plugins (with a few exceptions) are from Piwik.

@matt / @tsteur: Opinion?

@mattab commented on January 28th 2014 Owner

Nice one!

feedback:

  • I still see author_homepage and a couple occurences of 'author' in the code (via global search)
  • it's worth leaving 'By Piwik' otherwise it wouldn't be obvious; the only other indication is (core) but it's written in grey and people don't know what it means
@anonymous-piwik-user commented on January 28th 2014

In 3f6381b0e934e4f2c2e3eb0ce0dac3ed641e2402: Remove remaining occurences of author_home
Show Piwik for all core plugins

fixes #4578

@halfdan commented on January 28th 2014 Member

Done and good catch! :)

This Issue was closed on January 28th 2014
Powered by GitHub Issue Mirror