search

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

Refactored code base #57

Closed cmshawns closed 11 years ago

cmshawns commented 11 years ago

This pull request includes a refactored code base, resulting in more modularized code. It also includes changes previously submitted in pull request #54.

maricel0422 commented 11 years ago

@shawn-bellevuecollege

This pull request gives the following errors when activating the plugin:

Multisite clean installation Warning: include_once(/var/www/wpm/wp-admin) [function.include-once]: failed to open stream: Success in /var/www/wpm/wp-content/plugins/wordpress-cas-client/casManager.php on line 25

Warning: include_once() [function.include]: Failed opening '' for inclusion (include_path='.:/usr/share/php:/usr/share/pear') in /var/www/wpm/wp-content/plugins/wordpress-cas-client/casManager.php on line 25

The plugin generated 530 characters of unexpected output during activation. If you notice “headers already sent” messages, problems with syndication feeds or other issues, try deactivating or removing this plugin.

Single site Plugin could not be activated because it triggered a fatal error. Fatal error: Call to undefined function get_current_site() in /var/www/wp/wp-content/plugins/wordpress-cas-client/utilities.php on line 35

cmshawns commented 11 years ago

@maricel0422, I've pushed fixes to the errors you reported.

jonathanwindle-zz commented 11 years ago

This pull request has duplicate commits, 3fb753b, 8da6891, ef32966, and 854342d. I will attempt to rebase this pull request to clean up the commit log supporting the code changes before proceeding with testing.

jonathanwindle-zz commented 11 years ago

This pull request has been replaced by #58.