You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@cordova.apache.org by "Steve Gill (JIRA)" <ji...@apache.org> on 2014/08/01 00:01:39 UTC

[jira] [Commented] (CB-7227) cordova plugin add/rm should run plugman.js

    [ https://issues.apache.org/jira/browse/CB-7227?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14081559#comment-14081559 ] 

Steve Gill commented on CB-7227:
--------------------------------

Decided to leave it for now. We just added the conditionals instead. 

> cordova plugin add/rm should run plugman.js
> -------------------------------------------
>
>                 Key: CB-7227
>                 URL: https://issues.apache.org/jira/browse/CB-7227
>             Project: Apache Cordova
>          Issue Type: Improvement
>          Components: CLI, CordovaLib, Plugman
>            Reporter: Steve Gill
>
> cordova plugin add/rm directly calls plugman.raw.install/uninstall. It skips going through plugman.js and therefore https://github.com/surajpindoria/cordova-lib/blob/browserify/cordova-lib/src/plugman/plugman.js#L131 never runs.
> It should go through plugman.js. Seems like better design to me.



--
This message was sent by Atlassian JIRA
(v6.2#6252)