[wp-trac] Re: [WordPress Trac] #8568: Plugin activation hook should
not be run when scraping errors.
WordPress Trac
wp-trac at lists.automattic.com
Fri Jan 30 21:51:18 GMT 2009
#8568: Plugin activation hook should not be run when scraping errors.
------------------------------------------------------+---------------------
Reporter: sambauers | Owner: anonymous
Type: defect (bug) | Status: new
Priority: normal | Milestone: 2.7.1
Component: Administration | Version: 2.7
Severity: normal | Resolution:
Keywords: has-patch dev-feedback reporter-feedback |
------------------------------------------------------+---------------------
Comment (by hakre):
It is a recommended practise to place Upgrade, Cleanup, Install-Checks
etc. _in_ the Activation Hook [http://funcdoc.wordpress.com/2008/01/08
/writing-plugins-activation-and-deactivation/ 1].
The Request in Question is 'error_scrape'. I can not find a clear
description what this means and for what the code is intended for.
As long as it remains unclear wether or not the plugin shouldn't be
actived through 'error_scrape', the code should remain as it is and an
additional Hook should not be called.
Anyway, I adopt the criticism in this patch partly: It is simply unclear
what should happen here. Is the Plugin Activated anyway afterwards or is
this a Test _only_?
What about creating an additional Testdrive Process that is run only on
Activation?
--
Ticket URL: <http://trac.wordpress.org/ticket/8568#comment:7>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list