search

diydeveloper / cms-canvas

Laravel CMS
36 stars 33 forks source link

Update Common.php #15

Closed binaek closed 8 years ago

binaek commented 8 years ago

Fix for Notice: Only variable references should be returned by reference in Line 257 of System/Core/Common.php...

The issue arises in newer versions of PHP because In PHP assignment expressions always return the assigned value. So $_config[0] =& $config returns $config - but not the variable itself, but a copy of its value. And returning a reference to a temporary value wouldn't be particularly useful (changing it wouldn't do anything)

This fix has been merged in CI 2.2.1 https://github.com/bcit-ci/CodeIgniter/commit/69b02d0f0bc46e914bed1604cfbd9bf74‌​286b2e3 (January 2015)

Wmas commented 8 years ago

I had this issue to, but I already knew it so I have fix it. I am still asking for some help because, no matter what I try I still get this problem:

After successful installation I try to go to the login page but Firefox say: " The page isn't redirecting properly

Firefox has detected that the server is redirecting the request for this address in a way that will never complete. " Seems the author doesn't support this application anymore, but any help with this will be appreciated.

cosmomathieu commented 8 years ago

@Wmas did you delete or rename the installation folder?

Wmas commented 8 years ago

Thank you for your reply. Yes I have renamed the installation directory.

cosmomathieu commented 8 years ago

@Wmas what is your php version? Need to test to see how I can help. And you are correct about the lack of support, seems like he is busy working on a Laravel version.

Wmas commented 8 years ago

I have installed on my localhost with MAMP on Mac and I am running PHP 5.6.7

cosmomathieu commented 8 years ago

I am using WAMP and PHP 5.5.12 and my OS is Windows 10 but I think your redirect issue lies in your .htaccess file. I'm going to post troubleshooting questions/suggestions in issue #17