Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

Arkadiy

Гуру
5317
463 / 2
Крепитесь, други.

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #60 : 28.02.2012, 19:27:58 »

Цитата: Arkadiy от 28.02.2012, 19:21:03

В консоли в ответе пишет
Код
Notice: Constant JPATH_SITE already defined in W:\html\joostina14\www\administrator\index3.php on line 32
Код
Warning: session_start() [function.session-start]: Cannot send session cache limiter - headers already sent (output started at W:\html\joostina14\www\administrator\index3.php:32) in W:\html\joostina14\www\administrator\index3.php on line 43
Код
Warning: Cannot modify header information - headers already sent by (output started at W:\html\joostina14\www\administrator\index3.php:32) in W:\html\joostina14\www\administrator\index3.php on line 45
Получается что AJAX не проверяет исполнение запроса, раз уж при таком бардаке в ответе выдает что все сохранилось?

Эти убрал, у нас никто уже index3 не ползует, там два раза JPATH_SITE объявлен был.

Записан

А так ли проста Joomla? | Мои расширения для Joomla и Joostina

smart

Администратор
6478
1318 / 15
Хочешь сделать хорошо — сделай!

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #61 : 28.02.2012, 19:33:17 »

Хм, для совместимости с 1.0, у меня используется index3.php. Оставь его, просто оберни декларацию JPATH_SITE в проверку, как у вас в других местах реализовано.

p.s. используется в нескольких местах и было бы не удобно там добавлять лишние проверки

Записан

Новости Joomla | Joomla 3.x FAQ | JComments | Proofreader

smart

Администратор
6478
1318 / 15
Хочешь сделать хорошо — сделай!

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #62 : 28.02.2012, 19:35:53 »

Цитата: Arkadiy от 28.02.2012, 19:21:03

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

А зачем ему проверять - сообщение приходит с сервера, после того, как все сохранилось.

Записан

Новости Joomla | Joomla 3.x FAQ | JComments | Proofreader

smart

Администратор
6478
1318 / 15
Хочешь сделать хорошо — сделай!

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #63 : 28.02.2012, 19:48:11 »

Хм, я проверил - после того, как настройки сохраняются, кэш очищается:

Код: php

		$cache = JCommentsFactory::getCache('com_jcomments');
		$cache->clean();

Так что похоже надо разобраться, почему теряется разделитель в пути и получается cachecom_jcomments...

Судя по всему виной (\includes\libraries\cache\storage\file.php):

Код: php

$dir	= $this->_root . $folder;

Т.е. нет проверки, что в $this->_root в конце есть слеш. А эта переменная заполняется из переменной $options['cachebase'], которая, в свою очередь, инициализируется как:

Код: php

$this->_options['cachebase'] = JPATH_ROOT.DS.'cache';

Мне кажется, что надо просто поменять

Код: php

$dir	= $this->_root . $folder;

на

Код: php

$dir	= $this->_root . DS . $folder;

Иначе у вас любая именнованная группа кэша будет складываться в папку в корне сайта...

« Последнее редактирование: 28.02.2012, 19:52:58 от smart »

Записан

Новости Joomla | Joomla 3.x FAQ | JComments | Proofreader

Arkadiy

Гуру
5317
463 / 2
Крепитесь, други.

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #64 : 28.02.2012, 20:26:16 »

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

Записан

А так ли проста Joomla? | Мои расширения для Joomla и Joostina

Arkadiy

Гуру
5317
463 / 2
Крепитесь, други.

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #65 : 28.02.2012, 20:27:02 »

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

Такая-же ситуация с публикацией подписок.

« Последнее редактирование: 28.02.2012, 20:35:12 от Arkadiy »

Записан

А так ли проста Joomla? | Мои расширения для Joomla и Joostina

Arkadiy

Гуру
5317
463 / 2
Крепитесь, други.

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #66 : 28.02.2012, 21:00:32 »

С фронта два недочета:
1. Вместо даты возле имени комментирующего d.m.Y H:i
2. При клике на ид комментария выносит на главную с якорем #comment-2 (http://joostina14/#comment-2).
Все остальное работает.

Записан

А так ли проста Joomla? | Мои расширения для Joomla и Joostina

Gold Dragon

Захожу иногда
289
33 / 2

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #67 : 28.02.2012, 21:08:32 »

в "Дополнительные BBCode" в "Публикациях" у меня перебрасывает на "Комментарии", хотя "Кнопка" работает.. Вот только не понятно, почему с перезагрузкой, а не через AJAX? или так и задумано?

Код

$dir = $this->_root . DS . $folder;

Решило проблему с путями

Записан

smart

Администратор
6478
1318 / 15
Хочешь сделать хорошо — сделай!

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #68 : 28.02.2012, 22:01:01 »

Цитата: Arkadiy от 28.02.2012, 21:00:32

1. Вместо даты возле имени комментирующего d.m.Y H:i

Это проблема с языковым файлом, сейчас поправлю - там нужно сконвертировать формат, для 1.0 он конвертируется, а ветке для Joostina забыл.

Цитата: Arkadiy от 28.02.2012, 20:27:02

не работает изменение публикации из списка комментариев в админке при нажатии на иконку в строке комментария

Есть идеи? Там используется стандартный для 1.0 метод mosCommonHTML::PublishedProcessing, который в конечном счете использует JS функцию listItemTask.

Цитата: Gold Dragon от 28.02.2012, 21:08:32

Вот только не понятно, почему с перезагрузкой, а не через AJAX? или так и задумано?

Да я в административной панели AJAX почти не прикручивал еще.

Цитата: Arkadiy от 28.02.2012, 21:00:32

2. При клике на ид комментария выносит на главную с якорем #comment-2 (http://joostina14/#comment-2).

Не совсем понятно почему, надо смотреть...

Записан

Новости Joomla | Joomla 3.x FAQ | JComments | Proofreader

smart

Администратор
6478
1318 / 15
Хочешь сделать хорошо — сделай!

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #69 : 28.02.2012, 23:23:14 »

Arkadiy, Gold Dragon: я обновил дистрибутив, ссылка прежняя.

Решены следующие проблемы совместимости:

1. Формат даты и времени
2. Установка на свежие версии MySQL
3. Публикация работает в подписках, BBCode и т.д.

Тестировал пока на 1.3, но думаю на 1.4 эти пункты тоже будут работать. Очень хотелось бы наличия какой-то константы (чтобы ничего не подключать, не создавать объектов). На текущий момент приходится проверять наличие константы JPATH_BASE и _ISO2, но это не очень хорошо.

Записан

Новости Joomla | Joomla 3.x FAQ | JComments | Proofreader

Arkadiy

Гуру
5317
463 / 2
Крепитесь, други.

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #70 : 29.02.2012, 05:03:57 »

В Joostina есть класс joomlaVersion, в нем есть

Код

/** @var строка Продукт*/
	var $PRODUCT = 'Joostina';
	/** @var строка CMS*/
	var $CMS = 'Joostina';
	/** @var версия*/
	var $CMS_ver = '1.4.0beta';
	/** @var int Номер основной версии*/
	var $RELEASE = '1.0';
	/** @var строка  статус разработки*/
	var $DEV_STATUS = 'beta';
	/** @var int Подверсия*/
	var $DEV_LEVEL = '1400';
	/** @var int Номер сборки*/
	var $BUILD = '837';

Обращаться к переменным проще так - joomlaVersion::get('SVN')

Записан

А так ли проста Joomla? | Мои расширения для Joomla и Joostina

Gold Dragon

Захожу иногда
289
33 / 2

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #71 : 29.02.2012, 09:58:19 »

номер основной версии (витки) даст

Код

joomlaVersion::get('RELEASE');

в результате будет "1.3" или "1.4"

« Последнее редактирование: 29.02.2012, 10:12:20 от Gold Dragon »

Записан

smart

Администратор
6478
1318 / 15
Хочешь сделать хорошо — сделай!

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #72 : 29.02.2012, 11:05:35 »

Цитата: Arkadiy от 29.02.2012, 05:03:57

В Joostina есть класс joomlaVersion, в нем есть

Это все понятно, но как я раньше писал, константа (типа JVERSION в 1.5+) намного удобнее. Дело в том, что для JComments по сути не важно, Joostina 1.3 или 1.4, важно, что это Joostina, а не Joomla 1.0. И чтобы 100% без ошибок это сделать, мне нужно проверить, а есть ли файл с классом версии, подключить его, вызвать... короче, много лишних действий, а учитывая тот факт, что это нужно не только при установке, но и при работе - получается накладно.

По предыдущим ошибкам проверили - все исправлено?

Записан

Новости Joomla | Joomla 3.x FAQ | JComments | Proofreader

Arkadiy

Гуру
5317
463 / 2
Крепитесь, други.

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #73 : 29.02.2012, 11:17:04 »

Класс joomlaVersion подгружен всегда, по этому проверка будет не сложная

Код

if(class_exist('joomlaVersion') && joomlaVersion::get('CMS') == 'Joostina')

Если все-таки нужно константу, то сделаю. Новую сборку пока не тестировал, вечером займусь.

« Последнее редактирование: 29.02.2012, 11:20:46 от Arkadiy »

Записан

А так ли проста Joomla? | Мои расширения для Joomla и Joostina

smart

Администратор
6478
1318 / 15
Хочешь сделать хорошо — сделай!

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #74 : 29.02.2012, 11:25:34 »

Цитата: Arkadiy от 29.02.2012, 11:17:04

Класс joomlaVersion подгружен всегда, по этому проверка будет не сложная

А... вы его не переименовали? Я на автомате прочитал как joostinaVersion, поэтому и возник вопрос. Тогда все действительно проще.

Не, фигня. В оригинальном joomlaVersion нет метода get, так что если я его дерну, а расширение запущено на 1.0, получим ошибку и все упадет.

Записан

Новости Joomla | Joomla 3.x FAQ | JComments | Proofreader

Arkadiy

Гуру
5317
463 / 2
Крепитесь, други.

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #75 : 29.02.2012, 11:33:21 »

Тогда так: переменные в классе не защищены и с небольшим нарушением стандартов можно их вызвать без гет

Код

joomlaVersion::CMS

Так пойдет? Правда пока все чисто теоретичеки, но должно работать.

Записан

А так ли проста Joomla? | Мои расширения для Joomla и Joostina

smart

Администратор
6478
1318 / 15
Хочешь сделать хорошо — сделай!

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #76 : 29.02.2012, 11:34:28 »

Да не, лучше уж буду проверять JPATH_BASE + JTEMPLATE... По крайней мере warning'ов не будет по поводу доступа к non-static переменной

Записан

Новости Joomla | Joomla 3.x FAQ | JComments | Proofreader

Arkadiy

Гуру
5317
463 / 2
Крепитесь, други.

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #77 : 29.02.2012, 12:25:21 »

Ну и так можно, хотя стрикт стандарт врядли кто включает.

Записан

А так ли проста Joomla? | Мои расширения для Joomla и Joostina

Gold Dragon

Захожу иногда
289
33 / 2

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #78 : 29.02.2012, 12:26:41 »

Arkadiy, а что мешает сделать их статик? Не думаю что сильно затронит движок

Записан

Arkadiy

Гуру
5317
463 / 2
Крепитесь, други.

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #79 : 29.02.2012, 13:18:28 »

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

Записан

А так ли проста Joomla? | Мои расширения для Joomla и Joostina

smart

Администратор
6478
1318 / 15
Хочешь сделать хорошо — сделай!

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #80 : 29.02.2012, 13:24:10 »

Да нет, специально переделывать не стоит. Я просто поинтересовался нет ли более простого и универсального способа. Нет - ну значит нет. Жду какой-то информации по текущему состоянию вопроса совместимости.

Записан

Новости Joomla | Joomla 3.x FAQ | JComments | Proofreader

Arkadiy

Гуру
5317
463 / 2
Крепитесь, други.

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #81 : 29.02.2012, 14:31:14 »

При установке на 1.4.ошибка

Код

Fatal error: Call to undefined method DOMIT_Lite_TextNode::getAttribute() in W:\html\joostina14\www\administrator\components\com_installer\installer\installer.class.php on line 473
Call Stack
#	Time	Memory	Function	Location
1	0.0065	117312	{main}( )	..\index2.php:0
2	0.1961	4745792	require_once( 'W:\html\joostina14\www\administrator\components\com_installer\admin.installer.php' )	..\index2.php:97
3	0.2441	5692280	require( 'W:\html\joostina14\www\administrator\components\com_installer\installer\installer.php' )	..\admin.installer.php:32
4	0.2600	5726336	mosGeneralInstaller->uploadPackage( )	..\installer.php:34
5	6.1499	8391592	mosInstallerComponent->install( )	..\installer.php:159
6	10.5512	10341712	mosInstaller->parseFiles( )	..\component.class.php:123

Записан

А так ли проста Joomla? | Мои расширения для Joomla и Joostina

smart

Администратор
6478
1318 / 15
Хочешь сделать хорошо — сделай!

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #82 : 29.02.2012, 14:36:13 »

Ну вчера-то на 1.4 все ставилось, а я с тех пор вроде ничего в XML не менял. Да и падает-то оно у вас в установщике...

Записан

Новости Joomla | Joomla 3.x FAQ | JComments | Proofreader

Arkadiy

Гуру
5317
463 / 2
Крепитесь, други.

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #83 : 29.02.2012, 14:50:39 »

Это да, вчера я дома устанавливал, сейчас на работе, вроде везде Опенсервер стоит, Joostina свежую поставил, в установщике никто не рылся уже очень давно, попробовал другой компонент поставить - ставится без ошибок. Не понятно как-то.

Записан

А так ли проста Joomla? | Мои расширения для Joomla и Joostina

Arkadiy

Гуру
5317
463 / 2
Крепитесь, други.

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #84 : 29.02.2012, 15:23:53 »

Запустил на денвере, такая-же история, причем спотыкается инсталлер не сначала, фронтальные файлы заливает, а вот админские не хочет. Пойду домой, посмотрю как там получится.

Запустил установку на сервере, на денвере с джустиной 1.3.0.4, все то-же.

« Последнее редактирование: 29.02.2012, 15:36:35 от Arkadiy »

Записан

А так ли проста Joomla? | Мои расширения для Joomla и Joostina

smart

Администратор
6478
1318 / 15
Хочешь сделать хорошо — сделай!

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #85 : 29.02.2012, 16:01:59 »

А у меня на вчерашних 1.3 и 1.4 установился. Аркадий, я думаю стоит просто в отладке посмотреть, на каком именно теге спотыкается-то.

Записан

Новости Joomla | Joomla 3.x FAQ | JComments | Proofreader

Arkadiy

Гуру
5317
463 / 2
Крепитесь, други.

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #86 : 29.02.2012, 16:55:07 »

Я пытался научиться делать отладку, и н раз, но что-то у меня это не получилось, до сих пор пользуюсь var_dump, запускал и с ним, спотыкается в объекте DOMIT_Lite_Element, в котором есть этот метод, а не в DOMIT_Lite_TextNode в котором действительно нет.

Если ты напишешь статью или топик на форуме по организации отладки применительно к Joomla и PhpStorm я буду очень признателен, думаю и не я один. PhpStorm у меня есть 3 и 4 версии. На работе 2, комп слабенький, боюсь свежие ставить, хотя 4 поразила скоростью поиска по проекту, по сравенению с 3, а по сравнению с другими IDE любая за пояс заткнет.

Записан

А так ли проста Joomla? | Мои расширения для Joomla и Joostina

Arkadiy

Гуру
5317
463 / 2
Крепитесь, други.

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #87 : 29.02.2012, 17:10:19 »

Пришел домой. Вчерашний дистриб переименовал, сегодняшний скачал. Удалил вчерашний, устанавливаю сегодняшний, та же ошибка. Устанавливаю вчерашний - все на ура ставится. Парадокс.

Записан

А так ли проста Joomla? | Мои расширения для Joomla и Joostina

smart

Администратор
6478
1318 / 15
Хочешь сделать хорошо — сделай!

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #88 : 29.02.2012, 17:12:11 »

Слушай, ну сравни в том же WinMerge, чем они отличаются? Ну фактически в XML файле добавилось 2 строки, я добавил 2 картинки:

Код: xml

			<filename>assets/j_button1_prev.png</filename>
			<filename>assets/j_button1_right.png</filename>

Больше изменений в XML точно не было.

Записан

Новости Joomla | Joomla 3.x FAQ | JComments | Proofreader

Arkadiy

Гуру
5317
463 / 2
Крепитесь, други.

Re: Совместимость JComments 2.3 и Joostina 1.3.x (1.4.x)

« Ответ #89 : 29.02.2012, 17:58:33 »

Перезалил в сегодняшний дистриб XML от вчерашнего, не помогло, перепаковал сегодняшний, тоже не помогло. Ничего не понимаю, мистика какая-то.

Записан

А так ли проста Joomla? | Мои расширения для Joomla и Joostina

Jcomments & Joomla 3.5 & PHP 7 Автор ProtectYourSite	Ответов: 11 Просмотров: 6809	15.01.2024, 14:02:36 от Vastriet
Не отображаються комментарии на дополнительных языках Joomla 3.4.8 + Jcomments 3.0.5 Автор orsy	Ответов: 11 Просмотров: 4695	20.12.2021, 15:44:35 от igorsuccess
JComments 3.0.5 не отправляется коммент Автор EvaLifeTime	Ответов: 61 Просмотров: 10684	04.05.2021, 19:41:25 от ErikaMaza
Не отображаются лайки для голования в комментариях JComments 3.0.7 Автор vladankov	Ответов: 1 Просмотров: 2124	31.07.2019, 18:50:02 от denistischenko
Не отображается защитный код JComments (и не меняется на reCaptcha) Автор abbyevg	Ответов: 2 Просмотров: 1676	06.10.2018, 20:59:56 от abbyevg

Что нового в joomla 5.1?

Добавление полей Joomla в результаты Умного Поиска при помощи JFilters

Вышел JoomShopping 5.4.0

Похожие темы