Не могу переопределить функцию

Главные вкладки

Аватар пользователя molp molp 13 сентября 2012 в 15:55

В файле модуля Ubercart - uc_catalog - uc_catalog.module есть функция _uc_catalog_navigation
Внутри данной функции меня интересуют строки:

<?php
  
// No nodes in category or descendants. Not in path and display nothing.
  
if (!$num) {
    return array(
FALSE''0);
  }
?>

Если их удалить, то в блоке который выводит модуль будут отображаться все термины, вне зависимости от того, содержатся в них материалы или нет.
Но мы ведь учимся делать красиво, по этому пытаемся переопределить функцию в файле template.php

Вставляю в файл template.php все строки функции _uc_catalog_navigation. Саму функцию переименовываю в mytheme_uc_catalog_navigation и внутри уже делаю необходимые действия. Чищу кэш. Но ожидаемые изменения не вступают в силу.

Прошу помощи, сообщество!

Комментарии

Аватар пользователя Виктор Степаньков ака RxB Виктор Степаньк... 13 сентября 2012 в 15:57

Сообщество в моём лице вам хочет сказать о надобности чтения документации и изучения общего курса PHP.
Переопределяются функции темизации, ваша функция врядли таковой является

Аватар пользователя molp molp 13 сентября 2012 в 16:09

Спасибо за дельный совет. Как раз, вчера сделал себе подарок и приобрел несколько полезных книг по теме, но думаю что за пару дней никак не осилю.
По этому прошу подсказки, что нужно использовать для переопределения вышеназванной функции? Ubercart API?

Аватар пользователя sg85 sg85 14 сентября 2012 в 0:50

Как то стоял перед выбором переписывать с 0 модуль uc_file, либо сделать 2 патча по 2 строчки, угадайте что я выбрал?)

Правда через 3 месяца он обновился и 1 патч применился с ругательствами, другой отказался, благо есть привычка писать для себя readme с подробным описанием что где и зачем, пригодилось, ибо уже не помнил для чего они были нужны, не говоря про то куда вписывать

Аватар пользователя Виктор Степаньков ака RxB Виктор Степаньк... 14 сентября 2012 в 7:20

"sg85" wrote:
Как то стоял перед выбором переписывать с 0 модуль uc_file, либо сделать 2 патча по 2 строчки, угадайте что я выбрал?)

Разве кто-то говорит, что патчи это плохо?
Плохо это когда всё хакано перехакано бездумно, а потом ты берёшься за сайт, а он с приветом прошлого разработчика с приветом.

Аватар пользователя igorek igorek 14 сентября 2012 в 9:07

"sg85" wrote:
угадайте что я выбрал?

А я бы клонировал бы модуль, например uc_filemy положил бы в папку custom. И он уже не обновится...)))
Клонировать - это же пол часа...
У меня только с комментом за пол часа не получилось...)))

Аватар пользователя sg85 sg85 14 сентября 2012 в 12:39

Скриптом, если патчей больше 1, правда все равно не совсем удобно. По поводу клона, очень спорно, ибо обновления безопасности лишними назвать сложно, особенно в модуле самого магазина.