BellevueCollege / wordpress-cas-client

A CAS client plugin for Wordpress sites, forked from http://wordpress.org/support/plugin/wpcas-w-ldap
GNU General Public License v2.0
11 stars 8 forks source link

5 - Simplify plugin settings #19

Closed jonathanwindle-zz closed 11 years ago

jonathanwindle-zz commented 11 years ago

Screenshot of Plugin Configuration Page Take 2013-09-13 wordpress-cas-client-config

Currently the plugin configuration feels clunky. Settings should be simplified for easier readability and configuration.

Examples based on current configuration screen as of this writing:

TriptiSharma commented 11 years ago

Separating this "Help should be added for each parameter to explain the expected values." task, since its a general UI option for all the fields - #38

jonathanwindle-zz commented 11 years ago

The string for BaseDN may need to be converted to an all upper case string before saving to the database.

This is related to a bug experienced this morning where the BaseDN value equaled "DC=bellevuecollege,DC=edu". The value seemed to trigger a message stating that the WordPress CAS Client was not properly configured.

More testing is needed to properly evaluate if this is a bug that needs to be fixed for better user experience.

TriptiSharma commented 11 years ago

Please add instructions in Readme about the Default CAS File path - Issue #27

TriptiSharma commented 11 years ago

Task Notes:

  1. CAS.php Path - The first time a user activates the plugin, code will try to pre-populate this field by checking in this location dirname(FILE).'/CAS/CAS.php' NOTE - The task for the user to put CAS.php file in the above location should be completed in Issue #27
    If the file exists in that location the field is pre-populated with this path otherwise its left empty.
  2. "phpCAS::client() parameters" is replaced by CAS Server A new field CAS Server URI is created which replaces the 3 fields - Server hostname, Port and path. Code will parse the URI supplied by the user and will derive hostname, port and path from it.
  3. A new field 'LDAP URI' is created under LDAP parameters, which replaced LDAP hostname and LDAP Port. This field is parsed to retrieve hostname and port. If the user does not specifies a Port and Scheme (ldap:// or ldaps://) , the port is defaulted to 389 and the URI is appended with ldap://. If the port is not specified and the Scheme is ldaps://, Port is defaulted to 636. Port is not visible to user in this case on the settings page.