[3.1][3.2] Topic Solved

[rxu]

Отредактировал скрипт выше, попробуйте теперь.

[Sheer]

Ага, разница есть.
Рабочий

Код: Выделить всё

array(1) { ["ext/tierra/topicsolved/language/ru/info_acp_forums.php"]=> string(18) "tierra/topicsolved" }

array(2) {
  ["ext/tierra/topicsolved/language/ru/info_acp_forums.php"]=>
  string(18) "tierra/topicsolved"
  ["ext/tierra/topicsolved/language/ru/info_acp_styles.php"]=>
  string(18) "tierra/topicsolved"
}

Не рабочий

Код: Выделить всё

array(1) { ["ext/tierra/topicsolved/language/ru/info_acp_styles.php"]=> string(18) "tierra/topicsolved" }

array(2) {
  ["ext/tierra/topicsolved/language/ru/info_acp_styles.php"]=>
  string(18) "tierra/topicsolved"
  ["ext/tierra/topicsolved/language/ru/info_acp_forums.php"]=>
  string(18) "tierra/topicsolved"
}

[angst66]

У меня без вышеприведенных правок не подключается админка.

[rxu]

Sheer, первая строка - это обновленный скрипт дает?
Значит, куда-то отваливается info_acp_forums.php на нерабочем варианте.
Попробуй в конце еще добавить

Код: Выделить всё

var_dump($user_lang_files);
var_dump($english_lang_files);
var_dump($default_lang_files);

[Sheer]

angst66, Да это понятно что не подключается. Не понятно почему. Почему в одном случае подключается, в другом нет. В чем причина?

Попробуй в конце еще добавить

Чичиас

[angst66]

Код: Выделить всё

["ext/tierra/topicsolved/language/en/info_acp_styles.php"]=> string(18) "tierra/topicsolved" ["ext/tierra/topicsolved/language/en/info_acp_forums.php"]=> string(18) "tierra/topicsolved" } array(0) 

[Sheer]

info_acp_styles.php и info_acp_styles.php меняются местами
Рабочий

Код: Выделить всё

array(1) { ["ext/tierra/topicsolved/language/ru/info_acp_forums.php"]=> string(18) "tierra/topicsolved" }

array(2) {
  ["ext/tierra/topicsolved/language/ru/info_acp_forums.php"]=>
  string(18) "tierra/topicsolved"
  ["ext/tierra/topicsolved/language/ru/info_acp_styles.php"]=>
  string(18) "tierra/topicsolved"
}

array(2) {
  ["ext/tierra/topicsolved/language/ru/info_acp_forums.php"]=>
  string(18) "tierra/topicsolved"
  ["ext/tierra/topicsolved/language/ru/info_acp_styles.php"]=>
  string(18) "tierra/topicsolved"
}

array(2) {
  ["ext/tierra/topicsolved/language/en/info_acp_forums.php"]=>
  string(18) "tierra/topicsolved"
  ["ext/tierra/topicsolved/language/en/info_acp_styles.php"]=>
  string(18) "tierra/topicsolved"
}

array(0) {
}

Не рабочий

Код: Выделить всё

array(1) { ["ext/tierra/topicsolved/language/ru/info_acp_styles.php"]=> string(18) "tierra/topicsolved" }

array(2) {
  ["ext/tierra/topicsolved/language/ru/info_acp_styles.php"]=>
  string(18) "tierra/topicsolved"
  ["ext/tierra/topicsolved/language/ru/info_acp_forums.php"]=>
  string(18) "tierra/topicsolved"
}

array(2) {
  ["ext/tierra/topicsolved/language/ru/info_acp_styles.php"]=>
  string(18) "tierra/topicsolved"
  ["ext/tierra/topicsolved/language/ru/info_acp_forums.php"]=>
  string(18) "tierra/topicsolved"
}

array(2) {
  ["ext/tierra/topicsolved/language/en/info_acp_styles.php"]=>
  string(18) "tierra/topicsolved"
  ["ext/tierra/topicsolved/language/en/info_acp_forums.php"]=>
  string(18) "tierra/topicsolved"
}

array(0) {
}

Отправлено спустя 1 минуту 46 секунд:
angst66, я скрипт подправил малость - замени, а то результат нечитабельный.

[rxu]

Всё понятно, сейчас изложу причину.

Отправлено спустя 4 минуты 38 секунд:
Значит, фишка вот такая. Если экст содержит более 1-го файла модуля любого класса acp/ucp/mcp (или нескольких), проще говоря, больше одного файла вида info_Xcp_modulename.php, подключается только один, тот, что был считан из файловой системы первым. Почему не все? Поэтому (см. \includes\functions_module.php):

Код: Выделить всё

$lang_files = array_unique(array_merge($user_lang_files, $english_lang_files, $default_lang_files));

Функция array_unique оставляет только один файл в массиве, удаляя ключи с повторяющимися значениями имени экста, в данном случае tierra/topicsolved.
Надеюсь, не запутал.

[Sheer]

Не понятно тогда, почему в массив элементы попадают в разном порядке. От чего это зависит.
Короче, в сухом остатке, можно считать, что в расширении допущена ошибка, то есть не учтен тот факт, что

Функция array_unique оставляет только один файл в массиве, удаляя ключи с повторяющимися значениями имени экста

[rxu]

Если убрать array_unique, должно заработать как надо

Код: Выделить всё

$lang_files = array_merge($user_lang_files, $english_lang_files, $default_lang_files);

в расширении допущена ошибка

Скорее, это ошибка ядра.

[Sheer]

Сообщаем об ошибках разработчикам?

Отправлено спустя 3 минуты 18 секунд:

Если убрать array_unique

Подцепился английский.

[rxu]

Подцепился английский.

Видимо, просто последний из массива найденных файлов и подцепился (значения из русского перезаписаны).

[Anvar]

Должно стать нормально если использовать только один файл info_acp_*.php и не совсем понятно зачем в этом расширении два файла.

[Sheer]

Таки значит баг все же не в ядре, а в эксте.
Anvar, в этом расширении много чего непонятного. Никогда бы так делать не стал.

[rxu]

Anvar, так сработает, но концептуально это неверно. Экст может иметь более 1-го модуля администраторского/модераторского/личного разделов, и, соответственно, более 1-го файла для них, а подключится только один.