App\Model\Table\NotificationsTable

Complexity

Total Complexity

4

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

94.44%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	initialize()	0	7	1
A	addNotifications()	0	19	3

<?php

/* vim: set expandtab sw=4 ts=4 sts=4: */
/**
 * Notification model representing a notification for new report.
 *
 * phpMyAdmin Error reporting server
 * Copyright (c) phpMyAdmin project (https://www.phpmyadmin.net/)
 *
 * Licensed under The MIT License
 * For full copyright and license information, please see the LICENSE.txt
 * Redistributions of files must retain the above copyright notice.
 *
 * @copyright Copyright (c) phpMyAdmin project (https://www.phpmyadmin.net/)
 * @license   https://opensource.org/licenses/mit-license.php MIT License
 *
 * @see      https://www.phpmyadmin.net/
 */

namespace App\Model\Table;

use Cake\Model\Model;
use Cake\ORM\Table;
use Cake\ORM\TableRegistry;

/**
 * Notification Model.
 *
 * @property Developer $Developer
 * @property Report    $Report
 */
class NotificationsTable extends Table
{
    /**
     * Validation rules.
     *
     * @var array
     */
    public $validate = array(
        'developer_id' => array(
            'numeric' => array(
                'rule' => array('numeric'),
                'allowEmpty' => false,
                'required' => true,
            ),
        ),
        'report_id' => array(
            'numeric' => array(
                'rule' => array('numeric'),
                'allowEmpty' => false,
                'required' => true,
            ),
        ),
    );

    /**
     * The Associations below have been created with all possible keys,
     * those that are not needed can be removed.
     */

    /**
     * belongsTo associations.
     *
     * @var array
     */
    public function initialize(array $config)
    {
        $this->belongsTo('Reports', array(
            'className' => 'Reports',
            'foreignKey' => 'report_id',
        ));
    }

    /**
     * To Add Multiple Notifications for New report.
     *
     * @param int $report_id id of the new Report
     *
     * @return bool value. True on success. False on any type of failure.
     */
    public static function addNotifications($report_id)
    {
        if (!is_int($report_id)) {
            throw new InvalidArgumentException('Invalid Argument "$report_id"! Integer Expected.');
        }
        $devs = TableRegistry::get('Developers')->find('all');
        $notoficationTable = TableRegistry::get('Notifications');
        $res = true;
        foreach ($devs as $dev) {
            $notification = $notoficationTable->newEntity();
            $notification->developer_id = $dev['id'];

Accessing developer_id on the interface Cake\Datasource\EntityInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

            $notification->report_id = $report_id;

Accessing report_id on the interface Cake\Datasource\EntityInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

            $notification->created = date('Y-m-d H:i:s', time());

Accessing created on the interface Cake\Datasource\EntityInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

            $notification->modified = date('Y-m-d H:i:s', time());

Accessing modified on the interface Cake\Datasource\EntityInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

            $res = $notoficationTable->save($notification);
        }

        return $res;
    }
}