Cosmetic fixes

This commit is contained in:
Frederic Guillot 2015-10-19 21:32:18 -04:00
parent 7303a79b50
commit 039baacaa0
6 changed files with 13 additions and 7 deletions

View File

@ -2,6 +2,8 @@
namespace Kanboard\Model;
use PicoDb\Database;
/**
* Base model class
*
@ -20,7 +22,7 @@ abstract class Base extends \Kanboard\Core\Base
*/
public function persist($table, array $values)
{
return $this->db->transaction(function ($db) use ($table, $values) {
return $this->db->transaction(function (Database $db) use ($table, $values) {
if (! $db->table($table)->save($values)) {
return false;

View File

@ -4,6 +4,7 @@ namespace Kanboard\Model;
use SimpleValidator\Validator;
use SimpleValidator\Validators;
use PicoDb\Database;
/**
* Board model
@ -173,7 +174,7 @@ class Board extends Base
*/
public function saveColumnPositions(array $columns)
{
return $this->db->transaction(function ($db) use ($columns) {
return $this->db->transaction(function (Database $db) use ($columns) {
foreach ($columns as $column_id => $position) {
if (! $db->table(Board::TABLE)->eq('id', $column_id)->update(array('position' => $position))) {

View File

@ -2,6 +2,8 @@
namespace Kanboard\Model;
use use PicoDb\Database;
/**
* Project Daily Stats
*
@ -29,7 +31,7 @@ class ProjectDailyStats extends Base
{
$lead_cycle_time = $this->projectAnalytic->getAverageLeadAndCycleTime($project_id);
return $this->db->transaction(function ($db) use ($project_id, $date, $lead_cycle_time) {
return $this->db->transaction(function (Database $db) use ($project_id, $date, $lead_cycle_time) {
// This call will fail if the record already exists
// (cross database driver hack for INSERT..ON DUPLICATE KEY UPDATE)

View File

@ -2,6 +2,7 @@
namespace Kanboard\Model;
use PicoDb\Database;
use Kanboard\Event\SubtaskEvent;
use SimpleValidator\Validator;
use SimpleValidator\Validators;
@ -293,7 +294,7 @@ class Subtask extends Base
*/
public function savePositions(array $subtasks)
{
return $this->db->transaction(function ($db) use ($subtasks) {
return $this->db->transaction(function (Database $db) use ($subtasks) {
foreach ($subtasks as $subtask_id => $position) {
if (! $db->table(Subtask::TABLE)->eq('id', $subtask_id)->update(array('position' => $position))) {
@ -433,7 +434,7 @@ class Subtask extends Base
*/
public function duplicate($src_task_id, $dst_task_id)
{
return $this->db->transaction(function ($db) use ($src_task_id, $dst_task_id) {
return $this->db->transaction(function (Database $db) use ($src_task_id, $dst_task_id) {
$subtasks = $db->table(Subtask::TABLE)
->columns('title', 'time_estimated', 'position')