phpffcms / ffcms

Apps/Controller/Front/Profile.php

<?php

namespace Apps\Controller\Front;

use Apps\ActiveRecord\Profile as ProfileRecords;
use Apps\ActiveRecord\UserLog;
use Apps\ActiveRecord\UserNotification;
use Apps\Model\Front\Profile\FormSettings;
use Apps\Model\Front\Sitemap\EntityBuildMap;
use Extend\Core\Arch\FrontAppController;
use Ffcms\Core\App;
use Ffcms\Core\Exception\ForbiddenException;
use Ffcms\Core\Exception\SyntaxException;

/**
 * Class Profile. User profiles application front controller
 * @package Apps\Controller\Front
 */
class Profile extends FrontAppController
{
    const BLOCK_PER_PAGE = 10;
    const EVENT_CHANGE_PASSWORD = 'profile.changepassword.success';
    const NOTIFY_PER_PAGE = 25;
    const FEED_PER_PAGE = 10;
    const LOG_PER_PAGE = 5;

    /**
     * Fat actions like actionIndex(), actionShow() are located in standalone traits.
     * This feature allow provide better read&write accessibility
     */

    use Profile\ActionIndex {

The trait Apps\Controller\Front\Profile\ActionIndex requires the property $query which is not provided by Apps\Controller\Front\Profile.

        index as actionIndex;
    }

    use Profile\ActionShow {

The trait Apps\Controller\Front\Profile\ActionShow requires some properties which are not provided by Apps\Controller\Front\Profile: $role, $query

        show as actionShow;
    }

    use Profile\ActionFeed {

The trait Apps\Controller\Front\Profile\ActionFeed requires the property $query which is not provided by Apps\Controller\Front\Profile.

        feed as actionFeed;
    }

    use Profile\ActionWallDelete {

The trait Apps\Controller\Front\Profile\ActionWallDelete requires some properties which are not provided by Apps\Controller\Front\Profile: $target_id, $sender_id

        wallDelete as actionWalldelete;
    }

    use Profile\ActionAvatar {
        avatar as actionAvatar;
    }

    use Profile\ActionNotifications {

The trait Apps\Controller\Front\Profile\ActionNotifications requires the property $query which is not provided by Apps\Controller\Front\Profile.

        notifications as actionNotifications;
    }

    use Profile\ActionIgnore {

The trait Apps\Controller\Front\Profile\ActionIgnore requires the property $query which is not provided by Apps\Controller\Front\Profile.

        ignore as actionIgnore;
    }

    use Profile\ActionSearch {

The trait Apps\Controller\Front\Profile\ActionSearch requires the property $query which is not provided by Apps\Controller\Front\Profile.

        search as actionSearch;
    }

    use Profile\ActionUnblock {
        unblock as actionUnblock;
    }

    use Profile\ActionPassword {
        password as actionPassword;
    }


    /**
     * Show user messages (based on ajax, all in template)
     * @return string
     * @throws ForbiddenException
     */
    public function actionMessages()
    {
        if (!App::$User->isAuth()) {
            throw new ForbiddenException();
        }

        return $this->view->render('profile/messages');
    }

    /**
     * User profile settings
     * @return string
     * @throws \Ffcms\Core\Exception\SyntaxException
     * @throws ForbiddenException
     */
    public function actionSettings()
    {
        // check if auth
        if (!App::$User->isAuth()) {
            throw new ForbiddenException();
        }

        // get user object
        $user = App::$User->identity();
        // create model and pass user object
        $model = new FormSettings($user);

It seems like $user can also be of type null; however, parameter $user of Apps\Model\Front\Profile\FormSettings::__construct() does only seem to accept Ffcms\Core\Interfaces\iUser, maybe add an additional type check?

        // check if form is submited
        if ($model->send() && $model->validate()) {
            $model->save();
            App::$Session->getFlashBag()->add('success', __('Profile data are successful updated'));
        }

        // render view
        return $this->view->render('profile/settings', [
            'model' => $model
        ]);
    }

    /**
     * Show user logs
     * @return string
     * @throws ForbiddenException
     */
    public function actionLog()
    {
        // check if user is authorized
        if (!App::$User->isAuth()) {
            throw new ForbiddenException();
        }

        // get log records
        $records = UserLog::where('user_id', App::$User->identity()->getId());

        // build pagination info
        $totalCount = $records->count();
        $page = (int)$this->request->query->get('page', 0);
        $offset = $page * static::LOG_PER_PAGE;

        // apply pagination limits
        $records = $records->skip($offset)
            ->take(static::LOG_PER_PAGE)
            ->orderBy('id', 'DESC')
            ->get();

        // render output view
        return $this->view->render('profile/log', [
            'records' => $records,
            'pagination' => [
                'step' => static::LOG_PER_PAGE,
                'total' => $totalCount,
                'page' => $page
            ]
        ]);
    }

    /**
     * Cron schedule - build user profiles sitemap
     */
    public static function buildSitemapSchedule()
    {
        // get not empty user profiles
        $profiles = ProfileRecords::whereNotNull('nick');
        if ($profiles->count() < 1) {
            return;
        }

        // get languages if multilanguage enabled
        $langs = null;
        if (App::$Properties->get('multiLanguage')) {
            $langs = App::$Properties->get('languages');
        }

        // build sitemap from content items via business model
        $sitemap = new EntityBuildMap($langs);
        foreach ($profiles->get() as $user) {
            $sitemap->add('profile/show/' . $user->user_id, $user->updated_at, 'weekly', 0.2);
        }

        try {
            $sitemap->save('profile');
        } catch (SyntaxException $e) {

Consider adding a comment why this CATCH block is empty.

        }
    }

    /**
     * Cleanup tables as scheduled action
     */
    public static function cleanupTablesSchedule()
    {
        // calculate date (now - 1week) for sql query
        $date = (new \DateTime('now'))->modify('-1 week')->format('Y-m-d');
        try {
            UserNotification::where('created_at', '<=', $date)->delete();
            UserLog::where('created_at', '<=', $date)->delete();
        } catch (\Exception $e) {

Consider adding a comment why this CATCH block is empty.

        }
    }
}