phpffcms / ffcms

Apps/View/Front/default/widgets/newcontent/default.php

<?php
/** @var \Apps\ActiveRecord\Content[] $records */

use Ffcms\Core\Helper\Date;
use Ffcms\Core\Helper\Text;
use Ffcms\Core\Helper\Type\Str;

foreach ($records as $record) {
    $title = \App::$Translate->getLocaleText($record->title);
    if (Str::likeEmpty($title)) {
        continue;
    }
    $title = Text::snippet($title, 50);

It seems like $title can also be of type null; however, parameter $text of Ffcms\Core\Helper\Text::snippet() does only seem to accept string, maybe add an additional type check?

    $date = Date::humanize($record->created_at);
    $categoryUrl = \App::$Alias->baseUrl . '/content/list/' . $record->cpath;

The property cpath does not exist on Apps\ActiveRecord\Content. Did you mean path?

    $categoryLink = '<a href="' . $categoryUrl . '">' . \App::$Translate->getLocaleText($record->ctitle) . '</a>';

The property ctitle does not exist on Apps\ActiveRecord\Content. Did you mean title?

    $newsLink = \App::$Alias->baseUrl . '/content/read/' . $record->cpath;
    $newsLink = rtrim($newsLink, '/') . '/' . $record->path;

    echo '<div class="row"><div class="col-md-12">';
    echo '<a href="' . $newsLink . '">&rarr; ' . $title . '</a><br />';
    echo '<small class="float-left">' . $categoryLink . '</small>';
    echo '<small class="float-right text-secondary">' . $date . '</small>';

Are you sure $date of type false|string can be used in concatenation?

    echo '</div></div>';
    echo '<hr class="pretty" />';
}
?>