Wrap twitterservice's TwitterOauth vendor's OAuth include in class_exists()
|Reported by:||jricher||Owned by:|
|Severity:||major||Keywords:||oauth, plugin, crash, namespace|
Elgg 1.7.4 is incompatible with the community OAuth plugin, as the included libraries redefine several of the OAuth classes. If both plugins are enabled simultaneously, the site crashes.
Commenting out the call to include OAuth.php in the twitterOAuth.php file gets rid of the error, but I haven't done a full functionality test yet.
Ideally, the Twitter client should use the existing OAuth plugin for its library support. I don't see any reason in the code that it can't. I can look into what this would take and submit a patch back later, if there's interest in that solution.
Change History (5)
comment:1 Changed 3 years ago by jricher
- Difficulty set to moderate
- Milestone changed from Needs Review to Elgg 1.8
comment:4 Changed 2 years ago by brettp
- Difficulty changed from moderate to easy
- Summary changed from Elgg 1.7.4 twitterservice incompatible with OAuth plugin to Wrap twitterservice's TwitterOauth vendor's OAuth include in class_exists()