View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
12144 | Bug reports | Plugins | public | 2017-02-21 09:59 | 2017-02-23 11:45 |
Reporter | DSIN | Assigned To | ollehar | ||
Priority | none | Severity | partial_block | ||
Status | closed | Resolution | fixed | ||
Summary | 12144: Plugin AUTHCAS sous LimeSurvey 2.63 | ||||
Description | Hello, Here is the error message: Fatal error: Declaration of AuthCAS :: __ construct () must be compatible with plugins / plugins / AuthCAS / AuthCAS .php on line 421 | ||||
Tags | No tags attached. | ||||
Bug heat | 12 | ||||
Complete LimeSurvey version number (& build) | 2.63.0 (170304) | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | Firefox | ||||
Database type & version | MariaDB | ||||
Server OS (if known) | Linux CENTOS | ||||
Webserver software & version (if known) | apache | ||||
PHP Version | 5.6 | ||||
Looks like the LS release broke most of the plugins. |
|
thank you |
|
Please update to 2.63.1. |
|
I noticed it to... I managed to get it working again by changing:
into |
|
And that said... 170304 ? Should't it be 170220 ? |
|
The reason first version exploded was due to PluginManager class missing its namespace in __construct argument. But the change is reversed now, so should work in either case. |
|
The version date of release is wrong, but will be correct as soon as we pass into Mars. |
|
But to be on the save (future) side: |
|
save? safe :-) (one of these days!) |
|
Uh, we might or might not add that change again in 3.0.0, so yes, use init(). |
|
That's done than for us since this morning! Thnx! |
|
@asshank : did you make the pull request to the AuthCas github ? init is compatible in 2.06 and last 2.05 version. Not with the 1st 2.05 version, but someone using 1st 2.05 version didn't update a working plugin. 2.63.1 fix issue, but 3.0 => plugin api update |
|
@DenisChenu: I'm merely commenting on this ticket. I have a lot of plugins myselft that all user init() now. |
|
oops "DSIN" is the reporter! |
|
Yep, but AuthCas still on github no ? |
|
@DenisChenu: I didn't even knew such a plugin exists until this ticket. I made a contribution to this ticker because my plugings were also 'old fashion' |
|
@asshank : \o/ lol !! :)))) Maybe one of the first plugin we have : https://www.limesurvey.org/index.php?option=com_sobipro&pid=61&sid=85:AuthCAS&Itemid=729 |
|
I should have made it clear in my first note that I had to change my own plugins. |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2017-02-21 09:59 | DSIN | New Issue | |
2017-02-21 10:16 | jelo | Note Added: 43039 | |
2017-02-21 10:35 | DSIN | Note Added: 43040 | |
2017-02-21 13:44 | ollehar | Note Added: 43043 | |
2017-02-21 16:43 | asshank | Note Added: 43046 | |
2017-02-21 16:45 | asshank | Note Added: 43047 | |
2017-02-21 16:46 | ollehar | Note Added: 43048 | |
2017-02-21 16:47 | ollehar | Note Added: 43049 | |
2017-02-21 17:02 | asshank | Note Added: 43050 | |
2017-02-21 17:03 | asshank | Note Added: 43051 | |
2017-02-21 17:04 | ollehar | Note Added: 43052 | |
2017-02-21 17:07 | asshank | Note Added: 43053 | |
2017-02-21 19:04 | DenisChenu | Note Added: 43057 | |
2017-02-21 19:04 | DenisChenu | Issue Monitored: DenisChenu | |
2017-02-22 17:58 | asshank | Note Added: 43069 | |
2017-02-22 17:59 | asshank | Note Added: 43071 | |
2017-02-22 18:04 | DenisChenu | Note Added: 43072 | |
2017-02-22 18:23 | asshank | Note Added: 43075 | |
2017-02-22 18:26 | DenisChenu | Note Added: 43077 | |
2017-02-22 18:32 | asshank | Note Added: 43079 | |
2017-02-23 11:45 | ollehar | Assigned To | => ollehar |
2017-02-23 11:45 | ollehar | Status | new => closed |
2017-02-23 11:45 | ollehar | Resolution | open => fixed |
2022-02-10 10:41 | guest | Bug heat | 10 => 12 |