7

I've got a site on Codeigniter 2 and when I switch server's version to PHP7 I get the following two errors:

A PHP Error was encountered Severity: Notice Message: Only variables should be assigned by reference Filename: core/Controller.php Line Number: 51

$this->load->_base_classes =& is_loaded();

A PHP Error was encountered Severity: 8192 Message: Methods with the same name as their class will not be constructors in a future version of PHP; CI_DB_driver has a deprecated constructor Filename: database/DB_driver.php Line Number: 31

Does anyone know how to fix them?

mmatti
  • 319
  • 1
  • 2
  • 9
  • only remove change that line to $this->load->_base_classes =is_loaded(); and that should work fine :) – sdx11 Oct 31 '16 at 22:44

2 Answers2

13

Eventually I just updated CI core to CodeIgniter 2.2.6. Had to change DB driver to mysqli (since mysql is no longer supported in php7) and re-added the ci_sessions table in database (no idea why). And works like a charm!

mmatti
  • 319
  • 1
  • 2
  • 9
  • I am using PHP 7, Just updated my DB driver from `mysql` to `mysqli` and its worked – Qazi Apr 06 '17 at 09:46
  • Consider upgrading to CI3 as 2.2 has reached end-of-life some years ago. See other question's answer for details: https://stackoverflow.com/questions/41037434/codeigniter-compatibility-with-php-version – Roland Dec 22 '17 at 16:34
2

Only variables should be assigned by reference

This error isn't PHP 7-exclusive, you'd get it in older versions too. Anyway, I think the issue here is in is_loaded() and it is not returning a reference properly. Does it return by-reference (is it like function &is_loaded())? If not, it needs to. Does it return a variable or an expression? If it's not a variable, you need to put it in one before you can return a reference to it.

PHP manual page for this error: http://php.net/manual/en/language.references.return.php

Methods with the same name as their class will not be constructors in a future version of PHP; CI_DB_driver has a deprecated constructor

In PHP 4, you made a constructor method by naming it the same as the class. So if your class was class FooBar, your constructor would be public function FooBar. In PHP 5 and beyond, though, the recommended name for a constructor is __construct. So, go and edit that class and rename its constructor to get rid of the deprecation errors. Be sure to look at any extending classes to see if they call that constructor method, too, so you can change them.

See the upgrading guide: http://php.net/manual/en/migration70.deprecated.php

Also see the RFC: https://wiki.php.net/rfc/remove_php4_constructors

Andrea
  • 19,134
  • 4
  • 43
  • 65