Cotonti / Open Source PHP Content Management FrameworkContent Management Framework

Forums / National / Russian / Тестирование (Siena 0.9.2)

12>>>

Graber
#1 2011-05-18 13:34


Testers
Thanked: 5 times

Старая тема уж больно большая, открою новую.

Поставил вчера 0.9.2, что сразу бросилось в глаза:

cot_mail притерпела положительные изменения, но не все так гладко...

1) в теле письма портятся ссылки (возможно это дело рук вызова cot_title(..., ...) )
в ссылках "&" заменяется на "&" Попробуйте зарегистрироваться, или заказать восстановление пароля, и посмотрите ссылку.

2) Портится подпись. Почему-то все \n заменяются на \n\n
Как следвствие - получаем пустые строки в подписи.

Еще заметил, что в админке перестала работать кнопка "сброс настроек". (не знаю в какой версии сломалось)

Писал уже Трасту - видимо он забыл.
functions.php - cot_rc_output()
line 3678: $out['head_head'] .= $html;
именно эта строка портит жизнь при написании модулей. Пишу модуль, в модуле делаю вызов:

	cot_rc_link_file('http://ajax.googleapis.com/ajax/libs/jqueryui/1.8.11/jquery-ui.min.js');

естественно он не отрабатывает, т.к. сам jquery подключается уже после него. Лечится заменой 3678 строки на:

 

$out['head_head'] = $html.$out['head_head'];


Никаких проблем при этом не выявилось.

Кто бездействует, тот не завоевывает
Trustmaster
#2 2011-05-19 15:55


Administrators
Thanked: 191 times

Тикеты создал, спасибо!

May the Source be with you!
esclkm
#3 2011-05-22 16:06


Senior Developers
Thanked: 57 times

Мысли про ajax

Ajax необходимо изменить в сторону его упрощения.

  1. AjaxSend – является вспомогательной функцией, использоваие, которой в целом сложнеее, нежели $.ajax что следует учитывать
  2. Кастомные Ajax события на ошибку или при удачном выполнении такжне не нужны – причина изложена выше.
  3. В тоже время ajax хелперы оказались весьма удобными и практичными за исколючением истории, которую я бы заменил на более простую библиотеку (например http://plugins.jquery.com/project/history )
  4. То есть я предлагаю просто написать примитивных хелпер (без доп событий) который не будет использовать ajaxSend  (который в целом предлагаю утилизировать)
  5. При ошибке – не выдавать сообщение а перейти по основной ссылке.
  6. В модулях добавить опцию – включить выключить аякс.
  7. Начать делать для каждого модуля свой аякс. ( причина – комуто надо количество просмотров (например для банеров), а комуто удобство использования (количество уников то сохранится на прежнем уровне.
  8. Продолжу позже

Добавлено 1 часа спустя:

///

еще размышления про auth систему

права у нас нужны далеко не везде - например большая часть плагинов не ивспользует систему прав. втоже время система прав тянет на себя более 5 строк в auth таблице.  Мне кажется было бы весьма логично сделать возможность в setup файле указывать юзать или нет права.

littledev.ru - мой маленький зарождающийся блог о котонти.
снижение стоимости программирования и снижение стоимости производства разные вещи. Первое можно скорее сравнить с раздачей работникам дешевых инструментов, чем со снижением зарплаты

This post was edited by esclkm (2011-05-22 17:12, 2 years ago)
Alex300
#4 2011-05-24 09:48


Developers
Thanked: 22 times

свежая установка -  Warning: MemcachePool::set(): The lowest two bytes of the flags array is reserved for pecl/memcache internal use in /home/alexey/public_html/test_siena/public/system/cache.php on line 897

Приходится бежать со всех ног, чтобы только оставаться на месте. А чтобы куда-то попасть, приходится бежать еще быстрее...
...Sorry for my english...
Portal30.Ru
esclkm
#5 2011-05-24 13:28


Senior Developers
Thanked: 57 times

удаление плагина:

сообщение: ошибка: удалить? да вместе с данными.

А нет без данных нету...... и почему это ошибка?

littledev.ru - мой маленький зарождающийся блог о котонти.
снижение стоимости программирования и снижение стоимости производства разные вещи. Первое можно скорее сравнить с раздачей работникам дешевых инструментов, чем со снижением зарплаты
Trustmaster
#6 2011-05-24 17:30


Administrators
Thanked: 191 times

#29731 Alex300:

свежая установка -  Warning: MemcachePool::set(): The lowest two bytes of the flags array is reserved for pecl/memcache internal use in /home/alexey/public_html/test_siena/public/system/cache.php on line 897

В config.php надо отключить memcache - очистить значение $cfg['cache_drv'].

Added 2 minutes later:

#29732 esclkm:

удаление плагина:

сообщение: ошибка: удалить? да вместе с данными.

А нет без данных нету...... и почему это ошибка?

Потому что удаление без данных обычно имеет смысл в двух случаях: 1) для подключения новых частей или конфигов или удаления старых (то есть то, что делает Обновить); 2) для временного отключения (то есть то, что делает Приостановить).

Ошибкой сделано временно, пока нет универсального механизма подтверждения опасных действий.

May the Source be with you!
esclkm
#7 2011-05-25 09:59


Senior Developers
Thanked: 57 times

еще 2 вопроса. правда по url editor

1. когда переменная написана в середине строка то не срабаотывает простое указание ее во втором столбце а надо писать & :например не &d=*

2. часть переменных проскальзывают по всему движку например приведенная $d =прописывать ее всюду как не хочется. как ее прописать однократно - чтобы она применилась везде ко всем правилам.

Добавлено 1 недели спустя:

приостановка частей плагины вызывает отстановку всего плагина - транк

Добавлено 2 часа спустя:

в дебагере шаблонизатора баг - если в другом шаблоне был блок с таким же именен - то он больше выводиться не будет

littledev.ru - мой маленький зарождающийся блог о котонти.
снижение стоимости программирования и снижение стоимости производства разные вещи. Первое можно скорее сравнить с раздачей работникам дешевых инструментов, чем со снижением зарплаты

This post was edited by esclkm (2011-06-01 12:58, 2 years ago)
Graber
#8 2011-06-08 18:20


Testers
Thanked: 5 times

plugins/i18n/i18n.page.tags.php

Line 79-80: при определении PAGE_I18N_TRANSLATE и PAGE_I18N_TRANSLATE_URL вместо "=>" стоит ","

Кто бездействует, тот не завоевывает
Dayver
#9 2011-06-09 13:22


Developers
Thanked: 35 times

Удивило что в экстраполях (грубо говоря - новом функционале - поскольку код сильно отличается от того что был в ЭП в генуе) нет ни одного хука - той вещи которая делает легко расшыряемыми седитио и котонти

Pavel Tkachenko aka Dayver
Я злой и страшный серый волк, я в поросятах знааааюююю толк
esclkm
#10 2011-06-09 13:33


Senior Developers
Thanked: 57 times

экстра поля выполнены функциями. поясни зачем надо хук? я пока думал делать колбэк -поля - но есть ли от оных смысл??? при этом у нас хуки уже сейста практически через строчку.

littledev.ru - мой маленький зарождающийся блог о котонти.
снижение стоимости программирования и снижение стоимости производства разные вещи. Первое можно скорее сравнить с раздачей работникам дешевых инструментов, чем со снижением зарплаты
Alex300
#11 2011-06-09 14:45


Developers
Thanked: 22 times

#29821 esclkm:

я пока думал делать колбэк -поля - но есть ли от оных смысл??? при этом у нас хуки уже сейста практически через строчку.

Смысл есть, если, например, нужно экстраполе заполняемое значениями из БД, например выпадающий список категорий или выбор города.

Было бы здорово, если бы такие поля были в настройках модулей/плагинов. Описал в setup'e поле, указал какую функцию вызвать. Пользователь зашел в настройки плагина - это поле (список) автоматически заполнилось значениями из БД.

Приходится бежать со всех ног, чтобы только оставаться на месте. А чтобы куда-то попасть, приходится бежать еще быстрее...
...Sorry for my english...
Portal30.Ru
esclkm
#12 2011-06-09 15:26


Senior Developers
Thanked: 57 times

вообщщето callback тип настроек появился в 09. постмотрите внимательно на существующие в поставке и вы найдете примеры. адля кл бэк экстраполей надо как минимум 5-6 функций

littledev.ru - мой маленький зарождающийся блог о котонти.
снижение стоимости программирования и снижение стоимости производства разные вещи. Первое можно скорее сравнить с раздачей работникам дешевых инструментов, чем со снижением зарплаты
Dayver
#13 2011-06-11 23:30


Developers
Thanked: 35 times

При востановлении страницы из корзины имею белый экран и 

Fatal error: Call to undefined function cot_page_resync() in /modules/page/page.trashcan.php on line 35

Еще помню в генуа была ошибка которую вижу в сиене: когда просматриваешь права категории юзверей то в отображаемом списке категорий страниц ссылки все вида - /admin.php?m=page хотя должны быть типа - /page.php?c=blog , /page.php?c=system и т.п.

Pavel Tkachenko aka Dayver
Я злой и страшный серый волк, я в поросятах знааааюююю толк

This post was edited by Dayver (2011-06-12 00:37, 2 years ago)
Graber
#14 2011-06-18 21:45


Testers
Thanked: 5 times

Есть такое ощущение, что в плагине news есть одна неувязочка. В файле plugins/news/news.php после строки
/* === Hook - Part2 : Include === FIRST === */
нужно добавить еще пару строчек:
$news_join_columns = "";
$news_join_tables = "";
иначе при выводе новостей из более чем одной категории на главной могут возникнуть траблы. Как вариант - попробуйте плагин i18n+news (вывод более чем из одного раздела). Вместо i18n можно взять любой другой плагин, который дополняет эти 2 переменные.

Кто бездействует, тот не завоевывает
esclkm
#15 2011-06-19 08:58


Senior Developers
Thanked: 57 times

а что происходит там? ведь переменные то остаются одними и темеже? или просто они 2 раза наполняются?

littledev.ru - мой маленький зарождающийся блог о котонти.
снижение стоимости программирования и снижение стоимости производства разные вещи. Первое можно скорее сравнить с раздачей работникам дешевых инструментов, чем со снижением зарплаты

12>>>