livestreet / livestreet Goto Github PK

В панели добавления топика/комментария не подписана кнопка "Вставить юзера". Все остальные подписаны.

сделать плагин sitemap и admin по умолчанию

Couldn't find 'em. Where are the test cases located ?

При генерации ошибки 404 в /classes/actions/ActionError.class.php жестко указан протокол HTTP/1.1, хотя это может быть не так. В связи с этим, в некоторых связках серверов могут получаться ошибки, аналогичные, например, http://brainstorm.name/blog/nginx-apache-proxy-chunked-encoding-problem.html

Достаточно поменять в /classes/actions/ActionError.class.php

"HTTP/1.1 404 Not Found"

на

"{$_SERVER['SERVER_PROTOCOL']} 404 Not Found"

чтобы это пофиксить

Есть предложение сделать откружения/состояния проекта - development и production.

Создавать при этом отдельный конфиги config.development.php, config.production.php, в которых будут описаны такие параметры как кэширование данных(js,css,smarty templates) и другие какие заблагорассудятся.

Отличительной чертой production от development будет то что в production режиме мы будем использовать кэши созданные заранее, а в development будем создавать кэши ровным счетом как в текущей версии.

При этом будем создавать кэшированную версию конфигов, т.к на парсинг значений и поиск по ключам вида "path.root.server" уходит порядка 0,03 секунды из 0,08 рабочих. Кэшировать шаблоны в smarty навсегда(до перехода на development режим), а не с проверкой изменений в шаблоне(http://www.smarty.net/docsv2/ru/caching.tpl#caching.setting.up)

При составлении запросов UPDATE и DELETE в MapperORM при отсутствии у сущности primary_key, ри формировании данных для WHERE не происходит екранирование данных, что вызывает SQL error.

В данный момент отсутствует

при создании топика вылезает такая шняга
SQL Error: Unknown column 'comment_id' in 'field list' at Z:\home\beatsale.ls\www\classes\modules\comment\mapper\Comment.mapper.class.php line 163
Array ( [code] => 1054 [message] => Unknown column 'comment_id' in 'field list' [query] => SELECT comment_id, comment_id as ARRAY_KEY, comment_pid as PARENT_KEY FROM beatsale_comment WHERE target_id = 3 AND target_type = 'topic' ORDER by comment_id asc; [context] => Z:\home\beatsale.ls\www\classes\modules\comment\mapper\Comment.mapper.class.php line 163 )

Сейчас все методы кэшируются, кроме этого.
А он очень важен, так как через него получаются данные при вызове GetEntityNameBy.....
Этот метод в свою очередь вызывается связями HasOne и BelongsTo

Исправление тут kpoxas@a47d909

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

Только поставил. Все чистое. З даних только 1 адмін. Больше нигде ошибки не наблюдались

SQL Error: Unknown column 'c.comment_count_favourite' in 'field list' at /media/Garb/projects/workspace/classes/modules/comment/mapper/Comment.mapper.class.php line 668
Array ( [code] => 1054 [message] => Unknown column 'c.comment_count_favourite' in 'field list' [query] => UPDATE ls_comment c SET c.comment_count_favourite = ( SELECT count(f.user_id) FROM ls_favourite f WHERE f.target_id = c.comment_id AND f.target_publish = 1 AND f.target_type = 'comment' ) [context] => /media/Garb/projects/workspace/classes/modules/comment/mapper/Comment.mapper.class.php line 668 )

psnet@c224adf

не вызывается форма добавления поля, собственно ошибка

1. Создайте фотосет с некоторым описанием фотографий
2. Опубликуйте
3. Зайдите в редактирование, измените или добавьте подпись к фотографиям
4. После публикации вы увидите, что ничего не изменилось

1. вставляем изображение, окно не отцентрировано
2. отправляем пустое сообщение, получаем ошибки
   пишем в адресате свой ник, больше ничего не заполняем
   опять отправляем
   получаем 3 ошибки снова, поле "кому" очистилось
   надо не очищать поля при ошибках

Так конструкция в запросе SELECT выполняется быстрее, т.к. именно по тому полю,
по которому был поиск идет подсчет, а когда поле подсчета указывается явно, как это сделано
сейчас (count(user_id)), то после выборки по условиям мускул начинает
ещё выковыривать user_id для каждой строки и только тогда считать по
этому полю.

psnet@3526f01

При занесении в черновики топика, фотосета,... в экшенах вызывается метод

$this->Stream_write($oTopic->getUserId(), 'add_topic', $oTopic->getId(),$oTopic->getPublish() && $oBlog->getType()!='close');

Последним аргументом идет boolean тип.
То есть при сохранении в черновики опубликованного, всегда будет передаваться false.
В самом методе идет установка

$oEvent->setPublish($iPublish);

То есть при $iPublish = false в запросе мы получаем error:

SQL Error: Incorrect integer value: '' for column 'publish' at row 1 at ....\classes\modules\stream\mapper\Stream.mapper.class.php line 40
Array ( [code] => 1366 [message] => Incorrect integer value: '' for column 'publish' at row 1 [query] => UPDATE stream_event SET publish='' WHERE id = 3 [context] => ....\classes\modules\stream\mapper\Stream.mapper.class.php line 40 )

Для примера, попробуйте сохранить опубликованный фотосет в черновики.

Значение свойства $sMenuHeadItemSelect не влияет на его значение в шаблоне. К тому же, если закомментировать загрузку в шаблон этого свойства - также ничего не поменяется. В шаблоне все равно значение переменной $sMenuHeadItemSelect будет blogs

Фотосет: загружаем изображений больше, чем указано в конфиге для одновременной загрузки (к примеру в конфиге 20, загрузили 30). Начинаем просмотр. После показа 20й картинки показывается первая. Просмотр всех 30 изображений возможен только когда раскрыты все превьюшки.

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

Некорректно работает автоопределение primary key.
Наглядный пример. Есть таблица, без PRIMARY, но с ключами UNIQUE a_id и b_id.
При автоопределении PRIMARY по select columns оба эти поля возвращаются с aRow['Key'] == 'PRI', последнее из них и будет записана как '#primary_key', что приведет к ошибкам при работе с таблицей (столкнулся с этим при апдейте данных). Так же скорей всего при наличии реального PRIMARY и UNIQUE при не прописанном в сущности primary key, UNIQUE подменит его собой, хотя это не столь критично.

config.local загружается после инклюда файлов из папки include. Поэтому, если попытаться активировать хакер консоль в файле config.local, выбьет ошибку

Warning: call_user_func() expects parameter 1 to be a valid callback, class 'Debug_HackerConsole_Main' not found in \engine\include\function.php on line 63

То есть функция dump() будет существовать, а экземпляр класса Debug_HackerConsole_Main нет.

Добавить resize фото, загружаемое пользователем в профиль.

Could you please extend function buildFilter to make available to get topics by topic_type:

    if (isset($aFilter['topic_type'])) {
        $sWhere.=" AND t.topic_type =  '".(string)$aFilter['topic_type']."'"; 
    }   

It required by interested plugin but not available in core...

Thanks!

Функция pathinfo поразному ведет себя на платформах Windows/Linux

Array
(
[dirname] => /www/psychicblast/images/1
[basename] => my three girlfriends.jpg
[extension] => jpg
)

Array
(
[dirname] => /www/htdocs
[basename] => index.html
[extension] => html
[filename] => index
)

Из за этого у многих не загружаются изображения в топики и не отображаются загруженные изображения, т.к с использованием этой функции генерируются пути к изображениям. Хотелось бы обратить на данный момент внимание и привлечь к обсуждения. Проблема встретилась лично у меня при использовании ls на Windows.

Так же хочу отметить что версия php достаточно старая - 5.1.6. Поэтому стоит либо указать конкретно версию
http://livestreetcms.com/about/ , а не PHP5 или вставить проверку версии в инсталятор. Надеюсь проблему поставил, что то решим.

Для примера:
https://github.com/livestreet/livestreet/blob/master/classes/modules/topic/entity/TopicPhoto.entity.class.php : 27

Необходимы хуки в вьюшке просмотра топика, после <div class="content">

У положительного голоса за топик нет плюса:

При отрицательном голосе - минус появляется сразу после голоса и остается после перезагрузки страницы.
При положительном голосе - плюс появляется сразу после голоса, но пропадает после перезагрузки страницы.

в профиле дату рождения можно поставить 31 февраля

Чтобы оправдать возможности автокомплита методов в IDE.

public function AddFavouriteComment(ModuleFavourite_EntityFavourite $oFavourite) {
if( ($oFavourite->getTargetType()=='comment')
&& ($oComment=$this->Comment_GetCommentById($oFavourite->getTargetId()))
&& $oComment->getTargetType()=='topic') {
return $this->Favourite_AddFavourite($oFavourite);
}
return false;
}

тут стоит проверка на то к чему сделан комментарий. При добавлению комментариев к своей сущности оно не позволяет занести его в избранное. Предлагаю заменить
$oComment->getTargetType()=='topic'
на
in_array($oComment->getTargetType(), $types)
где $types - подтягивать из конфига массив разрешенных сущностей.

В маппере топиков, для выборок по фильтру аналогичных GetTopics() переделать двойной FROM на LEFT JOIN, добавить возможность JOIN другие таблицы, по аналогии с bulidFilter()

при удалении талков, содержание их и комментарии к ним остаются в БД. Иными словами - мертвый груз.

обновите пожалуйста смарти http://www.smarty.net/files/Smarty-3.1.6.tar.gz
много же фиксов прошло с момента последнего обновления

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

Реквестирую допиливание инсталлера, чтобы ЛС нормально ставился в PostgreSQL'ьную базу.

Здравствуйте, уважаемые разработчики.

Планируете ли вы добавить «Фотораму» в дефолтную сборку Livestreet?
Очень хорошая замена для существующей наработки фотосета.

P. S. Прости если задал свой вопрос в не правильном месте.

Нужны хуки в темплейты добавления/редактирования топиков вне <form>

ежели у нас по дефолту выставляется jquery-шаблон после инсталляции, то в плагине page шаблон default должен быть c панелью markitup, а не древней lspanel на mootools

По моему скромному мнению неплохо бы убрать ограничение на длину имени.
Ну или по крайней мере вместо
if (func_check(getRequest('profile_name'),'text',2,20)) {
сделать
if (func_check(getRequest('profile_name'),'text',1,255)) {

Ибо:

1. есть люди с именами (и даже фамилиями) длиной в одну в одну букву.
2. Есть люди (например я, LOL), у которых имя-фамилия длиннее 20 символов. Есть и длиннее 40. Неплохо бы либо убрать ограничение, либо сделать соответствующую настройку.
   А то дискриминация какая-то.

при голосовании в профиле счётчик проголосовавших не обновляется, только после рефреша сайта

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

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

Кто что думает на этот счет? Я думаю этот фичреквест как раз в тему готовящейся новой версии движка.

при получении исходного (оригинала) размера аватара блога getAvatarPath(0)

раньше при запросе исходного размера терялось расширение файла.

psnet@01e58d6

Когда впервие заходиш на страницу показивает. Потом пропадает. Если зделать имнеменения в шаблоне или активировать плагин - повторяетса.

Warning: chmod(): Operation not permitted in /media/Garb/projects/workspace/engine/lib/external/Smarty/libs/sysplugins/smarty_internal_write_file.php on line 50

Если на поиске просто нажать кнопку поиска, то выдаются результаты по слову «поиск». Это не правильно, кнопка не должны реагировать если в поле подсказка.

Не работает загрузка аватара и фотографии в профиле.
Отправляется input без выбранного изображения

На старых системах (FreeBSD 6.1) наблюдается проблема в метатегах - отдельные кириллические символы (х, я, Р и т. д.) отображаются некорректно. Например: "случайно обнаружил на простора� сети сайт курски� эникейщиков".

Для решения этой проблемы нужно добавить модификатор u в файл /engine/include/function.php строка 285
$aWords=preg_split("/\s+/",$sText); меняем на $aWords=preg_split("/\s+/u",$sText);

Предлагаю внести соответствующие изменения в упомянутый файл в репозитории. Спасибо за внимание.

Реквестирую нормальную админку вместо "четырёх ссылок". :)
Собственно, призываю комьюнити пообщаться на эту тему.

Кроме всего прочего — неплохо бы добавить чуть более информативный хелп (хоть бы и в тултипах). Например, неискушённому пользователю совсем не ясно, как составлять шаблоны для кастомных полей.

В добавок к первой части — совсем, имхо, не лишним было бы иметь возможность давать юзерам админские права прямо из админки/их профиля. А то лазить в базу за этим — как-то не комильфо...

Данный хук присутствует только в ActionTopic.class.php.

link,question,photoset остались почему то без него

добавьте пожалуйста к иконке избранного счётчик кол-ва добавивших к себе в избранное топик

In table prefix_topic for field topic_type change the type to varchar instead of enum.
This change will simplify the implementation of new types of topics.

Не работает свернуть / развернуть комментарии