search

webtechnick / CakePHP-Facebook-Plugin

CakePHP Facebook Plugin
http://facebook.webtechnick.com
445 stars 138 forks source link

Broken sessions fix #159

Closed wvdongen closed 10 years ago

wvdongen commented 10 years ago

Make sure CakeSession is used, otherwise CakePHP sessions will be ignored because the Facebook vendor lib will start a basic PHP session first.

wvdongen commented 10 years ago

To clarify this issue, see this commit https://github.com/webtechnick/CakePHP-Facebook-Plugin/commit/da0d300a5a7ab39c0da81b1a43aca764ba6472f2. It fixes the same issue. However, the Facebook SDK update after that commit have undone the fix. So the fix, which is not present anymore, was not correctly placed because it is undone after every SDK update.

webtechnick commented 10 years ago

This is solved in the latest version. Please update your master repository.

wvdongen commented 10 years ago

You fixed it in the wrong place. With every SDK update you will undo this fix. When you place the CakeSession start like it is in my commit, session _start() in the SDK will never run due to the if statements around it.

webtechnick commented 10 years ago

That fix doesn't work. It still throws the same error. I'm well aware I will have to alter the SDK every update.