shibboleth icon indicating copy to clipboard operation
shibboleth copied to clipboard

Update some WordPress API calls and some PHP 7 support

Open skoranda opened this issue 8 years ago • 2 comments

Added multi-site support for switch to blog functionality and updated calls to use the newer WordPress user object.

Replace deprecated get_usermetadata call with get_user_meta.

Check for index 'redirect_to' for $_REQUEST before calling shibboleth_session_initiator_url to avoid PHP notice.

Updated deprecated call update_usermeta to update_user_metada.

Replaced a call to the split() function with explode() since split is deprecated in PHP 5.3 and is no longer available in PHP 7.

skoranda avatar Jun 03 '16 13:06 skoranda

Looks good. I have a slight preference for !empty($var) over isset($var) && $var, but the effect is the same.

Note: This pull request would close #9 and close #15

jrchamp avatar Jun 03 '16 14:06 jrchamp

Hello, thank you for submitting this patch. I released version 1.8 today to resolve this and other issues and included a shoutout for your patch. I did not include the changes for "multi-site support for switch to blog functionality", as I was hoping you could provide further clarification on the purpose and use case that this is supporting. I am the new maintainer of the plugin and all further work on the plugin will be done in a new GitHub repository. If you believe that switch_to_blog change should still be included, can you please open up an issue/PR in the new repository with details on the use case? Thanks!

michaelryanmcneill avatar Aug 24 '17 00:08 michaelryanmcneill