Что не так с кодом PHP?

Question

[Марк Ешкилев]

Здравствуйте, подскажите что не так здесь..?

<?php
class Menu {
	private $menu;
	public function Menu() {
		$this->menu = array(
			'news' => array(
				'url'=>'/news',
				'title'=>_('Новости'),
				'active'=>false,
				'place' => 0,
				'parent' => false,
			),
			'wiki' => array(
				'url'=>'https://wiki.metrostroi.net" target="_blank',
				'title'=>'Wiki',
				'active'=>false,
				'place' => 0,
				'parent' => false,
			),
			'players' => array(
				'url'=>'/players',
				'title'=>_('Игроки'),
				'active'=>false,
				'place' => 0,
				'parent' => false,
			),
			'servers' => array(
				'url'=>'/servers',
				'title'=>_('Сервера'),
				'active'=>false,
				'place' => 0,
				'parent' => false,
			),
			'mag' => array(
				'url'=>'/mag',
				'title'=>'MAG',
				'active'=>false,
				'place' => 0,
				'parent' => false,
			),
			'admin' => array(
				'title' => _('ПУ'),
				'active' => false,
				'place' => 1,
				'parent' => false,
				'right' => 'adm',
				'icon' => 9,
			),
			'news_add' => array(
				'url'=>'/news_add',
				'title'=>_('Добавить новость'),
				'active'=>false,
				'place' => 0,
				'parent' => 'admin',
				'icon' => 9,
				'show_icon' => 'plus icon',
			),
			'admin_mag' => array(
				'title' => _('MAG: Жалобы'),
				'url'=>'/admin_mag',
				'active' => false,
				'place' => 1,
				'parent' => 'admin',
				'right' => 'mag_reports',
				'show_icon' => 'warning icon',
			),
			'admin_tickets' => array(
				'title' => _('Тикеты'),
				'url'=>'/admin_tickets',
				'active' => false,
				'place' => 1,
				'parent' => 'admin',
				'right' => 'tickets',
				'show_icon' => 'cube icon',
			),
			'admin_tests' => array(
				'title' => _('Тесты'),
				'url'=>'/admin_tests',
				'active' => false,
				'place' => 1,
				'parent' => 'admin',
				'right' => 'up_down',
				'show_icon' => 'check square icon',
			),
			'admin_log' => array(
				'title' => _('Журнал действий'),
				'url'=>'/admin_log',
				'active' => false,
				'place' => 1,
				'parent' => 'admin',
				'right' => 'log',
				'show_icon' => 'book icon',
			),
			'admin_questions' => array(
				'title' => _('Вопросы к тестам'),
				'url'=>'/admin_questions',
				'active' => false,
				'place' => 1,
				'parent' => 'admin',
				'right' => 'edit_tests',
				'show_icon' => 'table icon',
			),
		);
	}
	public function set_item_active ($id) {
		$this->menu[$id]['active'] = true;
		if ($this->menu[$id]['parent']) $this->set_item_active($this->menu[$id]['parent']);
	}
	private function show_item ($id, &$item, $kid = false) {
		global $logged_user;
		global $locale;
		$c = 0; $sub = '';
		if (isset($item['right']) and !($logged_user and $logged_user->take_group_info($item['right']) or (isset($item['icon']) and $logged_user and $logged_user->max_icon_id() >= $item['icon']))) {
			if (isset($item['icon']) and !($logged_user and $logged_user->max_icon_id() >= $item['icon']))
				return '';
			elseif (!isset($item['icon']))
				return '';
		}
		foreach ($this->menu as $tmp_id => $tmp_item)
			if ($tmp_item['parent'] and $tmp_item['parent'] == $id) {
				$sub .= $this->show_item($tmp_id, $tmp_item, true);
				if ($tmp_item['active'])
					$item ['active'] = true;
				$c++;
			}
		ob_start();
		include Base::PathTPL(($c)?($kid)?'menu/item_subdropdown':'menu/item_dropdown':'menu/item');
		return ob_get_clean();
	}
	public function show($tpl, $error = false) {
		global $logged_user;
		global $locale;
		$menu = array('','');
		foreach ($this->menu as $id => $item)
			if (!$item['parent'])
				$menu[$item['place']] .= $this->show_item($id, $item);
		ob_start();
		include Base::PathTPL('menu/'. $tpl);
		return ob_get_clean();
	}
}

ошибка:
[09-Jun-2021 21:46:54 Europe/Moscow] PHP Deprecated: Methods with the same name as their class will not be constructors in a future version of PHP; Menu has a deprecated constructor in A:\OpenServer\domains\millennium\classes\menu.class.php on line 2

Comments

[edward_freedom]

у тебя Menu функция или конструктор?

[Марк Ешкилев]

edward_freedom, вроде как конструктор

[edward_freedom]

Марк Ешкилев, шо

[Марк Ешкилев]

edward_freedom, извиняюсь, т9*

[edward_freedom]

Марк Ешкилев, я не особо слежу на новыми версиями пхп, но до 7 версии, конструкторы были такими https://www.php.net/manual/ru/language.oop5.decon.php

[Марк Ешкилев]

edward_freedom, хм, а если это функция, то тогда что делать, просто я сам немного не понимаю этот код, но дата его создания 2018

[edward_freedom]

Марк Ешкилев, назови ее именем, не совпадающим с именем класса

[Марк Ешкилев]

edward_freedom, окей, спасибо) А можете ещё подсказать почему у меня не хочет работать авторизация..?

public static function TakeAuth() {
		global $logged_user, $db;
		if(!isset($_COOKIE['mitrastroi_sid'])) {
			$logged_user = false;
			return;
		}
//		$db->execute("DELETE FROM `sessions` WHERE `session_date` < NOW() - INTERVAL 1 MONTH ");
		$user = new User($_COOKIE['mitrastroi_sid'], 'session_id');
		if($user->uid() <= 0) {
			$logged_user = false;
			return;
		}
		$logged_user = $user;
		$sessionID = Base::randString(128);
		$db->execute("UPDATE `sessions` SET `session_date`=NOW() WHERE `session_id`='{$db->safe($_COOKIE['mitrastroi_sid'])}'");
	}

Answer 1

[DevMan]

a что не понятно из сообщения?
в мохнатые годы конструктор (функция, которая автоматом вызывается при создании экземпляра класса) имел такое же название как и сам класс.
затем его отделили и сделали магический метод __construct.