pkp / orcidProfile

A plugin to pull ORCID information into a PKP user profile
GNU General Public License v3.0
16 stars 51 forks source link

ORCID plugin should not function without credentials entered #177

Closed librariam closed 3 years ago

librariam commented 3 years ago

Is your feature request related to a problem? Please describe. Here is the most common scenario and a source of 99% of ORCID-related tickets filed with PKP|PS:

Unfortunately I find that this initial experience sours the first impression and turns people away from using the plugin.

Describe the solution you'd like The bottom line is that the plugin should not be performing any of its functionality (particularly not direct author interactions) without being set up first. This follows the logic of #161

Some ideas for implementation:

asmecher commented 3 years ago

@withanage, this would be a good improvement to include in the next release.

withanage commented 3 years ago

@librariam @asmecher

sure, I will add the following settings into the next release.

@asmecher , is this generally possible currently for adding this kind of features when enabling plugins?

asmecher commented 3 years ago

@withanage, just off the top of my head, the PLN plugin had some conditions that were required in order for the plugin to be enabled: https://github.com/pkp/pln/blob/ojs-stable-3_1_2/PLNPlugin.inc.php#L375

withanage commented 3 years ago

Hi @librariam ,

I have implemented the plugin as per your definition, before releasing it and doing some more testing, I wanted to ask, if the implementation corresponds to your requirements?

Here is the workflow.

  1. At the beginning plugin is disabled.
  2. Settings is available also when the plugin is disabled.
  3. Status shows about the status of the plugin. (I will link the plugin doc here)
  4. With wrong.-credentials you cannot enable the plugin.
  5. With correct credentials you can enable the plugin.
  6. If globally configured the same rules for client validation apply and the save button is disabled.

One thing, which I could not add was a separate pop-up. But the plugin is not allowed to be enabled as per definition 2.

orcid_validation

librariam commented 3 years ago

Thank you @withanage . Just to clarify - what would be the behaviour if someone tries to enable the plugin without entering credentials?

withanage commented 3 years ago

@librariam , users cannot enable the plugin, until they enter credentials (valid) through the backend. They can click the enable plugin, but it will not enable it.

librariam commented 3 years ago

@withanage thank you, this is great! Not being able to enable the plugin without entering valid credentials is exactly what we're after!

withanage commented 3 years ago

@librariam

I have released this enhancements for

and integrated in the plugin-gallery.

asmecher commented 3 years ago

@withanage, is this issue ready to be closed?

withanage commented 3 years ago

@asmecher There was no bugs reported after adding the functionality in around 2 weeks. Yes, I will close it.