View Issue Details

This bug affects 1 person(s).
 12
IDProjectCategoryView StatusLast Update
12144Bug reportsPluginspublic2017-02-23 11:45
ReporterDSIN Assigned Toollehar  
PrioritynoneSeveritypartial_block 
Status closedResolutionfixed 
Summary12144: Plugin AUTHCAS sous LimeSurvey 2.63
Description

Hello,
I just installed LimeSurvey version 2.63 and the CAS plugin does not work.

Here is the error message:

Fatal error: Declaration of AuthCAS :: __ construct () must be compatible with plugins / plugins / AuthCAS / AuthCAS .php on line 421

TagsNo tags attached.
Bug heat12
Complete LimeSurvey version number (& build)2.63.0 (170304)
I will donate to the project if issue is resolvedNo
BrowserFirefox
Database type & versionMariaDB
Server OS (if known)Linux CENTOS
Webserver software & version (if known)apache
PHP Version5.6

Users monitoring this issue

DenisChenu

Activities

jelo

jelo

2017-02-21 10:16

partner   ~43039

Looks like the LS release broke most of the plugins.
https://www.limesurvey.org/forum/news/108424-limesurvey-2-63-0-build-170304-released#148908

DSIN

DSIN

2017-02-21 10:35

reporter   ~43040

thank you

ollehar

ollehar

2017-02-21 13:44

administrator   ~43043

Please update to 2.63.1.

asshank

asshank

2017-02-21 16:43

reporter   ~43046

I noticed it to... I managed to get it working again by changing:

public function __construct(PluginManager $manager, $id)   {
    parent::__construct($manager, $id);
    /**
     * Here you should handle subscribing to the events your plugin will handle
     */
 }        

into

 public function init() {
    /**
     * Here you should handle subscribing to the events your plugin will handle
     */
}
asshank

asshank

2017-02-21 16:45

reporter   ~43047

And that said... 170304 ? Should't it be 170220 ?

ollehar

ollehar

2017-02-21 16:46

administrator   ~43048

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.

ollehar

ollehar

2017-02-21 16:47

administrator   ~43049

The version date of release is wrong, but will be correct as soon as we pass into Mars.

asshank

asshank

2017-02-21 17:02

reporter   ~43050

But to be on the save (future) side:
Use public function init()?

asshank

asshank

2017-02-21 17:03

reporter   ~43051

save? safe :-) (one of these days!)

ollehar

ollehar

2017-02-21 17:04

administrator   ~43052

Uh, we might or might not add that change again in 3.0.0, so yes, use init().

asshank

asshank

2017-02-21 17:07

reporter   ~43053

That's done than for us since this morning!

Thnx!

DenisChenu

DenisChenu

2017-02-21 19:04

developer   ~43057

@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

asshank

asshank

2017-02-22 17:58

reporter   ~43069

@DenisChenu: I'm merely commenting on this ticket. I have a lot of plugins myselft that all user init() now.
Problem solved. I thnk the Original jelo is the one using it!

asshank

asshank

2017-02-22 17:59

reporter   ~43071

oops "DSIN" is the reporter!

DenisChenu

DenisChenu

2017-02-22 18:04

developer   ~43072

Yep, but AuthCas still on github no ?

asshank

asshank

2017-02-22 18:23

reporter   ~43075

@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'

DenisChenu

DenisChenu

2017-02-22 18:26

developer   ~43077

@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

asshank

asshank

2017-02-22 18:32

reporter   ~43079

I should have made it clear in my first note that I had to change my own plugins.

Issue History

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