2016-08-14 109 views
0

首先,如果标题有点模糊,但是这是我能想到的最好的。我的问题: 我有几个任务,每个任务可以有子任务。 创建任务时,它将在recursiveParentUpdater()方法中设置DB中的finished_at字段。出于某种原因,我保存后的新任务是他自己的父(但不在数据库中,仍为NULL)。

例如,

$task = new Task; 
// set my properties 
$task->save(); // let's say ID = 5 

$task->parent->parent->parent->...->id === 5 // I have no idea why this happens 

然后,$ task-> recursiveParentUpdater()使用$ this-> parent获取自身并将其全部拧紧。 只有在创建新任务时才会发生这种情况,当删除不再有$任务给父母时。

它全部用于正常工作(本地和共享主机),2天后未触及它我遇到过这个问题(只在共享主机上,本地仍然没有问题)。 我的Debian机器仍在运行PHP5.6,共享主机支持5.6和7,但它不会改变。

我的数据库架构被更新(增加了 '显示创建表的任务')

Schema::create('tasks', function (Blueprint $table) { 
     $table->increments('id'); 
     $table->string('title'); 
     $table->string('description')->nullable(); 
     $table->integer('parent_id')->unsigned()->nullable(); 
     $table->integer('user_id')->unsigned()->nullable(); 
     $table->date('deadline_date')->nullable()->default(null); 
     $table->time('deadline_time')->nullable()->default(null); 
     $table->datetime('finished_at')->nullable()->default(null); 
     $table->timestamps(); 

     $table->foreign('parent_id')->references('id')->on('tasks')->onUpdate('cascade')->onDelete('cascade'); 
     $table->foreign('user_id')->references('id')->on('users')->onUpdate('cascade')->onDelete('cascade'); 
    }); 

or 

tasks| CREATE TABLE `tasks` (
`id` int(10) unsigned NOT NULL AUTO_INCREMENT, 
`title` varchar(255) COLLATE utf8_unicode_ci NOT NULL, 
`description` varchar(255) COLLATE utf8_unicode_ci DEFAULT NULL, 
`parent_id` int(10) unsigned DEFAULT NULL, 
`user_id` int(10) unsigned DEFAULT NULL, 
`deadline_date` date DEFAULT NULL, 
`deadline_time` time DEFAULT NULL, 
`finished_at` datetime DEFAULT NULL, 
`created_at` timestamp NULL DEFAULT NULL, 
`updated_at` timestamp NULL DEFAULT NULL, 
PRIMARY KEY (`id`), 
KEY `tasks_parent_id_foreign` (`parent_id`), 
KEY `tasks_user_id_foreign` (`user_id`), 
CONSTRAINT `tasks_parent_id_foreign` FOREIGN KEY (`parent_id`) REFERENCES `tasks` (`id`) ON DELETE CASCADE ON UPDATE CASCADE, 
CONSTRAINT `tasks_user_id_foreign` FOREIGN KEY (`user_id`) REFERENCES `users` (`id`) ON DELETE CASCADE ON UPDATE CASCADE 
) ENGINE=InnoDB AUTO_INCREMENT=337 DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci | 

我的模型(部分是相关)

public function user() { 
    return $this->belongsTo('App\User'); 
} 

public function children() { 
    return $this->hasMany('App\Task', 'parent_id', 'id'); 
} 

public function parent() { 
    return $this->belongsTo('App\Task'); 
} 

public function hasParent() { 
    return (!is_null($this->parent)) ? true : false; 
} 

public function hasChildren() { 
    return (count($this->children) > 0) ? true : false; 
} 

public function updateParentFinishedStatus() { 
    function recursiveParentUpdater($task) { 
     if ($task->hasParent()) { 
      $task->parent->setFinishedAt(date('d-m-Y H:i:s')); 

      foreach ($task->parent->children as $child) { 
       if (!$child->isFinished()) { 
        $task->parent->setFinishedAt(null); 
        break; 
       } 
      } 
      $task->parent->save(); 

      if ($task->parent->hasParent()) recursiveParentUpdater($task->parent); 
     } 
    } 

    recursiveParentUpdater($this); 
} 

我控制器

public function postCreate(Request $request) { 
    $this->validate($request, [ 
     'title' => 'string|required', 
     'description' => 'string', 
     'ptid' => 'integer|exists:tasks,id', 
     'deadline_date' => 'date|after:yesterday|required_with:deadline_time', 
     'deadline_time' => 'dateformat:H:i', 
    ]); 

    $task = new Task; 
    $task->setTitle($request->get('title')); 
    $task->setDescription($request->get('description')); 
    $task->setDeadline($request->get('deadline_date'), $request->get('deadline_time')); 

    if (!empty($request->get('ptid'))) { 
     $task->parent()->associate($request->get('ptid')); 
    } 

    $task->user()->associate(Auth::user()); 

    $task->save(); 

    $task->updateParentFinishedStatus(); 

    return redirect()->back(); 
} 

public function getDelete(Request $request) { 
    $this->validate($request, [ 
     'tid' => 'integer|required|exists:tasks,id', 
    ]); 

    $task = Task::where('id', $request->get('tid'))->where('user_id', Auth::user()->id)->first(); 

    $task->delete(); 

    $task->updateParentFinishedStatus(); 

    return redirect()->back(); 
} 
+0

你应该为你的表作为SQL,而不是提供实际的架构,包括相关的数据,使我们可以尝试重现该问题,但到目前为止,它听起来像是将'parent_id'设置为新创建的任务的'id'。 – Sherif

回答

0

我仍然不确定为什么它是主机上的问题,而不是我的本地机器上,但这是固定的。
我改变了这行代码

public function hasParent() { 
    return (!is_null($this->parent)) ? true : false; 
} 

public function hasParent() { 
    return (!is_null($this->parent_id)) ? true : false; 
}