search

hydrian / TTRSS-Auth-LDAP

GitHub repository for Tiny Tiny RSS's auth_ldap plugin
https://github.com/hydrian/TTRSS-Auth-LDAP/wiki
Other
28 stars 21 forks source link

Not compatible with new versions of tt-rss #2

Closed kamikazejoe closed 10 years ago

kamikazejoe commented 11 years ago

Plugin stopped working after upgrading tt-rss from 1.7.8 to 1.8. After copying directory for plugin into the 1.8 plugins directory, tt-rss reports this error:

E_USER_WARNING (512) classes/pluginhost.php:141 Plugin auth_ldap is not compatible with current API version (need: 2, got: 1)

hydrian commented 11 years ago

Ok. I haven't upgraded to 1.8 yet so I haven't seen this error yet. I'll setup a 1.8 environment to test it.

hydrian commented 11 years ago

This has been fixes in trunk. I'm going to rework the logging before the releasing a final release.

1of16 commented 11 years ago

you maybe should add these lines ad the end, so it works with tt-rss :) function api_version() { return 2; }

and there are some strange messages in the system-log: E_STRICT (2048) usr/share/php/Net/LDAP2/Schema.php:493 Non-static method PEAR::isError() should not be called statically, assuming $this from incompatible context E_STRICT (2048) plugins/auth_ldap/init.php:197 Non-static method PEAR::isError() should not be called statically, assuming $this from incompatible context E_STRICT (2048) usr/share/php/Net/LDAP2/Filter.php:295 Non-static method PEAR::isError() should not be called statically

hydrian commented 11 years ago

I already added the API function. I must have forgot to push it. I will look into that.
The isError method is odd. What version of PEAR::Net_LDAP2 are you using?

1of16 notifications@github.com wrote:

you maybe should add these lines ad the end, so it works with tt-rss :) function api_version() { return 2; }

and there are some strange messages in the system-log: E_STRICT (2048) usr/share/php/Net/LDAP2/Schema.php:493 Non-static method PEAR::isError() should not be called statically, assuming $this from incompatible context E_STRICT (2048) plugins/auth_ldap/init.php:197 Non-static method PEAR::isError() should not be called statically, assuming $this from incompatible context E_STRICT (2048) usr/share/php/Net/LDAP2/Filter.php:295 Non-static method PEAR::isError() should not be called statically

Reply to this email directly or view it on GitHub: https://github.com/hydrian/TTRSS-Auth-LDAP/issues/2#issuecomment-21407881

Sent from my Android phone with K-9 Mail. Please excuse my brevity.

1of16 commented 11 years ago

2.0.12, the last one you can get: http://pear.php.net/package/Net_LDAP2

hydrian commented 11 years ago

The isError() warning messages are addressed in issue #4

hydrian commented 10 years ago

The isError is a Net_LDAP2 issue not an auth_ldap issue. I've taken out all of the isError() method calls and it is still throwing "Non-static method PEAR::isError() should not be called statically". I must be happening internally. If you don't want to see those errors, I could turn off E_STRICT logging in PHP. It often doesn't affect production. It is good to know in development though.