search

craftsmancoding / repoman

Repoman allows for easy package development in MODX Revolution. Take back the simplicity!
30 stars 7 forks source link

Changing PluginEvents then importing should remove old events #7

Closed fireproofsocks closed 10 years ago

fireproofsocks commented 10 years ago

To reproduce:

  1. Create a plugin with a docblock identifying one event

    /**

    • @name Example
    • @PluginEvents OnWebPagePrerender */
  2. Run repoman import
  3. Change the docblock to specify a different event, e.g.
    • @PluginEvents OnLoadWebDocument
  4. Run repoman import again

Expected behavior is that the plugin is associated with only the latest event (OnLoadWebDocument). Actual behavior is that the plugin is associated with both events (OnLoadWebDocument and OnWebPagePrerender)

fireproofsocks commented 10 years ago

026754c..2deadb1 master -> master