$commenter->sendNotifications();
$report->draft_comment->delete();
\XF::session()->reportLastRead = \XF::$time; protected function finalizeReportComment(\XF\Service\Report\Commenter $commenter)
{
$commenter->sendNotifications();
$report = $commenter->getReport();
$report->draft_comment->delete();
$this->session()->reportLastRead = \XF::$time;
}$warningBy = $this->warningBy; $commenter = $this->service('XF:Report\Commenter', $report);
$commenter->setMessage(null);
$commenter->setReportState('resolved', $warningBy);
$commenter->save();.потому что жалоба закрыта и логично, что если она закрыта, то там черновик не нужен
Благодарю =)
Файл
XF\Service\User\Warn protected function _save()
{
if ($this->warning->isUpdate())
{
throw new \LogicException("This warning has already been saved");
}
$warning = $this->warning;
$warning->save();
if ($this->conversationTitle)
{
$this->sendConversation($warning);
}
if ($this->contentAction)
{
$this->handler->takeContentAction(
$this->content,
$this->contentAction,
$this->contentActionOptions
);
}
return $warning;
}Серьёзно?))) Я должен был придумать свой велосипед? Или тупо изменить статус entity? И тут опять же вопрос касается совместимости с другими дополнениями, которые могут что-то делать при закрытии жалобы.
Копипаст движка.
Ну значит модераторы не знают правил и такие модераторы даром не сдались.
Блэт, серьёзно? Такое чувство, что там будет по 10к жалоб на одно сообщение. По такой же схеме можно прикопаться к кэшу бб-кодов и прочего. Их может быть миллионы. Ужас.
public function findActiveBbCodes()
{
return $this->finder('XF:BbCode')
->where('active', 1)
->whereAddOnActive()
->setDefaultOrder('bb_code_id');
}
public function getBbCodeCacheData()
{
$bbCodes = $this->findActiveBbCodes()->fetch();
$cache = [];
foreach ($bbCodes AS $bbCodeId => $bbCode)
{
$bbCode = $bbCode->toArray();
unset($bbCode['bb_code_id'], $bbCode['active'], $bbCode['addon_id']);
$cache[$bbCodeId] = $bbCode;
}
return $cache;
}Так у того, у кого нет, тот и не ставит аддон :ROFLMAO:
Интересно узнать, по какому принципу выдавались оценочные баллы))
.как это связано вообще с этим? Оно не работает в каком-то месте?)) Или появляется супер-нагрузка?))
Так значит если так делают разрабы это правильно, если кто то другой это нет?
.что? Я что-то вообще не понял, к чему это было написано.
Его там и не было, мы текст руками не писали.
Базовые основы... Разработчики в приведенном примере так сделали, потому что у них
$warning используется потом несколько раз, для сохранения и передачи, там действительно имеет смысл выделить в отдельную переменную, но не для однократного использования.Как и говнокодеры, но это совсем другая история.
Можно, наверное даже нужно - и разработчики уровня Xon и прочих стараются такие моменты учитывать.
Ты пишешь аддон для неопределенного круга лиц - может и будет.
Да попытки доебки еще и не было, тут базовые знания хромают, чего там что-то более сложное описывать.
.а если он там был? Черновика нет и если мы просто закрываем жалобу, но разрабы его удаляют
Мне больше нравится такой вариант, он никак не влияет на производительность.
:giggles:
Когда будет, тогда можно будет говорить, а пока это "если бы да кабы"
Я не увидел ни одного аргумента, который бы это показал. :)
Собственно что и требовалось доказать. Когда-нибудь понимание придет, жаль что не сегодня.
