phpffcms / ffcms

Apps/View/Admin/default/comments/delete.php

<?php
use Ffcms\Core\Helper\Date;
use Ffcms\Core\Helper\Simplify;
use Ffcms\Core\Helper\Type\Str;
use Ffcms\Templex\Url\Url;

This use statement conflicts with another class in this namespace, Url. Consider defining an alias.

/** @var \Ffcms\Templex\Template\Template $this */
/** @var Apps\Model\Admin\Comments\FormCommentDelete $model */
/** @var string $type */

$this->layout('_layouts/default', [
    'title' => __('Delete comments'),
    'breadcrumbs' => [
        Url::to('main/index') => __('Main'),
        Url::to('widget/index') => __('Widgets'),
        Url::to('comments/index') => __('Comments'),
        __('Delete comments and answers')
    ]
]);
$records = $model->getRecord();
?>

<?php $this->start('body') ?>

<?= $this->insert('comments/_tabs') ?>

<h1><?= __('Delete comments and answers') ?></h1>
<?= __('Are you sure to delete this comments or answers?') ?>

<?php
$table = $this->table(['class' => 'table table-striped'])
    ->head([
        ['text' => '#'],
        ['text' => __('Message')],
        ['text' => __('Author')],
        ['text' => __('Date')]
    ]);
foreach ($records as $item) {
    $table->row([
        ['text' => $item->id],
        ['text' => Str::sub(\App::$Security->strip_tags($item->message), 0, 50)],

It seems like App::Security->strip_tags($item->message) can also be of type array and null; however, parameter $string of Ffcms\Core\Helper\Type\Str::sub() does only seem to accept string, maybe add an additional type check?

        ['text' => Url::a(['user/update', [$item->user_id]], Simplify::parseUserNick($item->user_id, $item->guest_name)), 'html' => true],

It seems like Ffcms\Core\Helper\Simplify::parseUserNick($item->user_id, $item->guest_name) can also be of type null; however, parameter $text of Ffcms\Templex\Url\Url::a() does only seem to accept string, maybe add an additional type check?

        ['text' => Date::convertToDatetime($item->created_at, Date::FORMAT_TO_HOUR)]
    ]);
}

?>

<div class="table-responsive"><?= $table->display() ?></div>

<?php $form = $this->form($model) ?>
<?= $form->start() ?>

<?= $form->button()->submit(__('Delete'), ['class' => 'btn btn-danger'])?>

<?= $form->stop() ?>

<?php $this->stop() ?>