plugins icon indicating copy to clipboard operation
plugins copied to clipboard

Plugins are available with "wrong" names

Open ralsina opened this issue 9 years ago • 6 comments

For example, the local_search plugin is available in the site as localsearch.

So:

nikola plugin -i localsearch

versus:

nikola plugin -r local_search

One or the other needs changing, or we need aliases so things work.

ralsina avatar Jun 03 '16 15:06 ralsina

Just listing down other problematic names:

rest_microdata -- microdata

rest_forms -- forms

local_search -- localsearch

render_mustache -- mustache

render_subindexes -- subindexes

tx3TagCloud -- tx3_tag_cloud

wordpress -- wordpress_compiler

punchagan avatar Jul 01 '16 09:07 punchagan

Just a remark: since both names of the WordPress compiler appear in the import_wordpress plugin coming with Nikola, a simple rename would be a bad idea as it could break released Nikola versions (if they are used to import a WordPress blog with conversion of posts to HTML enabled). So an alias mechanism might be a good idea.

felixfontein avatar Jul 02 '16 18:07 felixfontein

@felixfontein I don't understand your comment. The wordpress importer ships with core, so you never install it or remove it.

ralsina avatar Sep 14 '16 14:09 ralsina

The WordPress importer allows to automatically install the WordPress compiler plugin when asked to do so. If we change the name of the WordPress compiler plugin, existing installations of Nikola won't find the plugin anymore if that functionality of the import_wordpress plugin is used.

felixfontein avatar Sep 15 '16 00:09 felixfontein

No problem, as long as the name we keep is the one used to install :-)

On Wed, Sep 14, 2016 at 9:07 PM Felix Fontein [email protected] wrote:

The WordPress importer allows to automatically install the WordPress compiler plugin when asked to do so. If we change the name of the WordPress compiler plugin, existing installations of Nikola won't find the plugin anymore if that functionality of the import_wordpress plugin is used.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/getnikola/plugins/issues/149#issuecomment-247194336, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAGK4d0Lw9R06C3MIJYrZdbTXrxY8QXks5qqIxPgaJpZM4Itob- .

ralsina avatar Sep 15 '16 00:09 ralsina

Well, that's part of the problem: the import plugin uses both names: one (wordpress) to find the plugin (to determine if it is installed, and to use it when it is installed), and one (wordpress_compiler) to install it (doing so automatically if requested, or by printing the command to install it).

felixfontein avatar Sep 15 '16 01:09 felixfontein