Почему переменная пустая?

Question

[Влад]

У меня есть во такой кусок кода из Datalife.
При обращении к данной функции переменная $device оказывается пустой..
Поясните, пожалуйста, что это за запись $device = $this->desktop; ? Я слаб в php, что такое $this -> desktop? почему он пустой?

function check_device( $matches=array() ) {

		$block = $matches[2];
		$device = $this->desktop;

		if ($matches[1] == "smartphone" OR $matches[1] == "tablet" OR $matches[1] == "desktop") $action = true; else $action = false;
		if ($matches[1] == "smartphone" OR $matches[1] == "not-smartphone") $device = $this->smartphone;
		if ($matches[1] == "tablet" OR $matches[1] == "not-tablet") $device = $this->tablet;

		if( $action ) {

			if( !$device ) return "";

		} else {

			if( $device ) return "";

		}

		return $block;
	}

============================
опишу подробнее:
У меня дле 9.5, я пытаюсь внедрить кусок код от дле 10.6, а именно [tablet][/tablet], внутри которого должен отображаться текст только на планшетах.

В шаблоне я написал [tablet]тест[/tablet]
Проблема: этот текст у меня спрятан на всех устройствах, т.к. при обращении к function check_device переменным назначаются следующие значения:
$action = true
$device = false
а по логике кода при данных параметрах должен отдаваться return ""

И это на всех устройствах!

Я не могу понять, КАКОЙ КУСОК КОДА отвечает за определение устройства, с которого просматривается страница: комп, планшет, мобила??

https://github.com/Gordondalos/union/blob/da06053e...

Comments

[Дмитрий Кравченко]

приведите полный код. check_device это метод какого-то класса. если в поле desktop экземпляра этого ничего не положить, то оно будет пустым, это естественно

[Михаил]

Дмитрий Кравченко: https://github.com/Gordondalos/union/blob/da06053e... наверное код тут

[Влад]

да, https://github.com/Gordondalos/union/blob/da06053e... правильный файл.
почему при обращении к функции echo $device ничего не выдаёт?

[Михаил]

Влад: потому что $device == false

[Влад]

Михаил: да, действительно.. но почему не работает тег [tablet]...

опишу подробнее:
У меня дле 9.5, я пытаюсь внедрить кусок код от дле 10.6, а именно [tablet][/tablet], внутри которого должен отображаться текст только на планшетах.

В шаблоне я написал [tablet]тест[/tablet]
Проблема: этот текст у меня спрятан на всех устройствах, т.к. при обращении к function check_device переменным назначаются следующие значения:
$action = true
$device = false
а по логике кода при данных параметрах должен отдаваться return ""

И это на всех устройствах!

Я не могу понять, КАКОЙ КУСОК КОДА отвечает за определение устройства, с которого просматривается страница: комп, планшет, мобила??

https://github.com/Gordondalos/union/blob/da06053e...

Answer 1

[Михаил]

из доки
В пределах методов класса доступ к нестатическим свойствам может быть получен с помощью -> (объектного оператора): $this->property

Comments

[Влад]

не очень понимаю.. если вы разбираетесь, можете подсказать как можно ошибку выявить?

[Влад]

с каким именно классом идёт работа в данном куске кода?

[Влад]

где ссылка на этот класс?

[Михаил]

Влад: Какую ошибку? и check_device это не функция, а метод класса dle_template

[Игорь Воротнёв]

> с каким именно классом идёт работа в данном куске кода?
С текущим, тем в котором находится этот код (о чем говорит переменная $this)

[Михаил]

Игорь Воротнёв: https://github.com/Gordondalos/union/blob/da06053e...

[Игорь Воротнёв]

Михаил: Строка 24, class dle_template

[Игорь Воротнёв]

Михаил: $this->desktop - это обращение к переменной $desktop внутри текущего класса dle_template (сокращенная форма обращения к нему - $this). Свойство содержит булевое значение - true по умолчанию, может содержать и false. Что вы подразумеваете под "пустая переменная"? Какое значение вы там ожидаете увидеть?

[Влад]

Игорь Воротнёв: спасибо! у меня действительно не была задана переменная $desktop - добавил её.

Проблема осталась - опишу подробнее:
У меня дле 9.5, я пытаюсь внедрить кусок код от дле 10.6, а именно [tablet][/tablet], внутри которого должен отображаться текст только на планшетах.

В шаблоне я написал [tablet]тест[/tablet]
Проблема: этот текст у меня спрятан на всех устройствах, т.к. при обращении к function check_device переменным назначаются следующие значения:
$action = true
$device = false
а по логике кода при данных параметрах должен отдаваться return ""

И это на всех устройствах!

Я не могу понять, КАКОЙ КУСОК КОДА отвечает за определение устройства, с которого просматривается страница: комп, планшет, мобила??

https://github.com/Gordondalos/union/blob/da06053e...

[Игорь Воротнёв]

Влад: За определение отвечаюь строчки 44-55, внутри метода __construct. То есть, при инициализации (вызове) класса dle_template, данный метод вызывается автоматически. В свою очередь, в строке 44 данный метод инициализирует другой класс, Mobile_Detect. Именно в следующих строчках происходит проверка того, что вернул данный класс (какие свойства в нем и с какими значениями) и устанавливаются свойства текущего класса в зависимости от свойств класса Mobile_Detect.

[Влад]

Игорь Воротнёв: а у меня совсем отсутствовал function __construct ! не заметил, что и его надо перенести с дле 10.6 в дле 9.5. СПАСИБО!

вижу, что правильно определяются телефон и компьютер.
а вот планшет считает телефоном.. буду разбираться почему так..

СПАСИБО ВАМ! сложно помочь, когда я так плохо объясняю проблему...

[Михаил]

Влад: ну так у тебя и возвращается пустая строка вместо блока.
устройство определяют методы объекта Mobile_Detect: isMobile и isTablet в конструкторе класса dle_template (от 44 строки примерно)

[Влад]

Михаил: да, у меня эти строки отсутствовали в файле.. поставил и всё заработало! спасибо большое за помощь! удачи вам!! :)

[Игорь Воротнёв]

Влад: На здоровье)