Ноль фото в фотосете

если поставить в конфиге:
$config['module']['topic']['photoset']['count_photos_min'] = 0;


… то при попытке сохранить фотосет без загруженных фототграфий выдает:
Notice: Undefined offset: 0 in /home/delfi171/public_html/betshot.ru/classes/actions/ActionPhotoset.class.php on line 492

Fatal error: Call to a member function getId() on a non-object in /home/delfi171/public_html/betshot.ru/classes/actions/ActionPhotoset.class.php on line 494


p.s.: А почему пользователь обязан загружать фото? Не лучше при отсутствии загруженных фото просто не выводить блок с фотографиями в фотосете? как это допилить??

6 комментариев

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

почему не объеденить эти сущности? (и опрос туда же можно:) Есть загруженнные фотки — вот тебе фотосет, нет — вот тебе топик!

+ при редактировании темплэйтов приходится топик и фотосет отдельно редактировать (тут, конечно, и плюсы и минусы есть).
avatar
почему не объеденить эти сущности?
были такие вопросы ещё когда он в 0.5 версии появился.
avatar
строчки 491-:-494
if (!($oPhotoMain=$this->Topic_getTopicPhotoById(getRequestStr('topic_main_photo')) and $oPhotoMain->getTargetTmp()==$sTargetTmp)) {
 $oPhotoMain=$aPhotos[0];
}
$oTopic->setPhotosetMainPhotoId($oPhotoMain->getId());
взять в условие
if(count($aPhotos)){...}

но это на глаз, не исключено наличие более глубоких граблей… надо смотреть более внимательно.
  • SVK
  • 0
Только зарегистрированные и авторизованные пользователи могут оставлять комментарии.