Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

remove outdated code from Example* plugins #4209

Closed
mattab opened this issue Oct 9, 2013 · 3 comments
Closed

remove outdated code from Example* plugins #4209

mattab opened this issue Oct 9, 2013 · 3 comments
Assignees
@tsteur
Labels
Bug For errors / faults / flaws / inconsistencies etc. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical.
Milestone
2.0

Comments

@mattab
Copy link
Member

mattab commented Oct 9, 2013

The plugin ExamplePlugin Controller index() contains some outdated code.
This was used to generate the page: http://piwik.org/docs/plugins/functions/

What should we do with it?

Can this plugin be used as the code generator template?

This ticket goal is to remove outdated code from Example* plugins.
@tsteur
Copy link
Member

tsteur commented Oct 31, 2013

In b2bcbbe: refs #4209 removed deprecated example plugin

@tsteur
Copy link
Member

tsteur commented Oct 31, 2013

In 905a0ca: refs #4209 renamed ExamplePluginTemplate -> ExamplePlugin

@tsteur
Copy link
Member

tsteur commented Oct 31, 2013

In 7d03638: refs #4209 fix some tests because we added an API to examplePlugin

@mattab mattab added this to the 2.0 - Piwik 2.0 milestone Jul 8, 2014
sabl0r pushed a commit to sabl0r/piwik that referenced this issue Sep 23, 2014
@tsteur
refs matomo-org#4209 removed deprecated example plugin
b2bcbbe
sabl0r pushed a commit to sabl0r/piwik that referenced this issue Sep 23, 2014
@tsteur
refs matomo-org#4209 renamed ExamplePluginTemplate -> ExamplePlugin
905a0ca
sabl0r pushed a commit to sabl0r/piwik that referenced this issue Sep 23, 2014
@tsteur
refs matomo-org#4209 fix some tests because we added an API to exampl…
7d03638 
…ePlugin
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tsteur tsteur

Labels
Bug For errors / faults / flaws / inconsistencies etc. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical.
Projects
None yet
Milestone
2.0
Development

No branches or pull requests
2 participants
@tsteur @mattab