2008-07-25 08:26:05 +00:00
|
|
|
<?php
|
|
|
|
/* -*- tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
|
|
|
|
/*
|
|
|
|
# ***** BEGIN LICENSE BLOCK *****
|
|
|
|
# This file is part of InDefero, an open source project management application.
|
|
|
|
# Copyright (C) 2008 Céondo Ltd and contributors.
|
|
|
|
#
|
|
|
|
# InDefero is free software; you can redistribute it and/or modify
|
|
|
|
# it under the terms of the GNU General Public License as published by
|
|
|
|
# the Free Software Foundation; either version 2 of the License, or
|
|
|
|
# (at your option) any later version.
|
|
|
|
#
|
|
|
|
# InDefero is distributed in the hope that it will be useful,
|
|
|
|
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
|
|
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
|
|
# GNU General Public License for more details.
|
|
|
|
#
|
|
|
|
# You should have received a copy of the GNU General Public License
|
|
|
|
# along with this program; if not, write to the Free Software
|
|
|
|
# Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
|
|
|
|
#
|
|
|
|
# ***** END LICENSE BLOCK ***** */
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Update an issue.
|
|
|
|
*
|
|
|
|
* We extend IDF_Form_IssueCreate to reuse the validation logic.
|
|
|
|
*/
|
|
|
|
class IDF_Form_IssueUpdate extends IDF_Form_IssueCreate
|
|
|
|
{
|
|
|
|
public $issue = null;
|
|
|
|
|
|
|
|
public function initFields($extra=array())
|
|
|
|
{
|
|
|
|
$this->user = $extra['user'];
|
|
|
|
$this->project = $extra['project'];
|
|
|
|
$this->issue = $extra['issue'];
|
|
|
|
if ($this->user->hasPerm('IDF.project-owner', $this->project)
|
|
|
|
or $this->user->hasPerm('IDF.project-member', $this->project)) {
|
|
|
|
$this->show_full = true;
|
|
|
|
}
|
|
|
|
if ($this->show_full) {
|
|
|
|
$this->fields['summary'] = new Pluf_Form_Field_Varchar(
|
|
|
|
array('required' => true,
|
|
|
|
'label' => __('Summary'),
|
|
|
|
'initial' => $this->issue->summary,
|
|
|
|
'widget_attrs' => array(
|
|
|
|
'maxlength' => 200,
|
|
|
|
'size' => 67,
|
|
|
|
),
|
|
|
|
));
|
|
|
|
}
|
|
|
|
$this->fields['content'] = new Pluf_Form_Field_Varchar(
|
|
|
|
array('required' => false,
|
|
|
|
'label' => __('Comment'),
|
|
|
|
'initial' => '',
|
|
|
|
'widget' => 'Pluf_Form_Widget_TextareaInput',
|
|
|
|
'widget_attrs' => array(
|
|
|
|
'cols' => 58,
|
|
|
|
'rows' => 9,
|
|
|
|
),
|
|
|
|
));
|
|
|
|
if ($this->show_full) {
|
|
|
|
$this->fields['status'] = new Pluf_Form_Field_Varchar(
|
|
|
|
array('required' => true,
|
|
|
|
'label' => __('Status'),
|
|
|
|
'initial' => $this->issue->get_status()->name,
|
|
|
|
'widget_attrs' => array(
|
|
|
|
'maxlength' => 20,
|
|
|
|
'size' => 15,
|
|
|
|
),
|
|
|
|
));
|
|
|
|
$initial = ($this->issue->get_owner() == null) ? '' : $this->issue->get_owner()->login;
|
|
|
|
$this->fields['owner'] = new Pluf_Form_Field_Varchar(
|
|
|
|
array('required' => false,
|
|
|
|
'label' => __('Owner'),
|
|
|
|
'initial' => $initial,
|
|
|
|
'widget_attrs' => array(
|
|
|
|
'maxlength' => 20,
|
|
|
|
'size' => 15,
|
|
|
|
),
|
|
|
|
));
|
|
|
|
$tags = $this->issue->get_tags_list();
|
|
|
|
for ($i=1;$i<7;$i++) {
|
|
|
|
$initial = '';
|
|
|
|
if (isset($tags[$i-1])) {
|
|
|
|
if ($tags[$i-1]->class != 'Other') {
|
|
|
|
$initial = (string) $tags[$i-1];
|
|
|
|
} else {
|
|
|
|
$initial = $tags[$i-1]->name;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
$this->fields['label'.$i] = new Pluf_Form_Field_Varchar(
|
|
|
|
array('required' => false,
|
|
|
|
'label' => __('Labels'),
|
|
|
|
'initial' => $initial,
|
|
|
|
'widget_attrs' => array(
|
|
|
|
'maxlength' => 50,
|
|
|
|
'size' => 20,
|
|
|
|
),
|
|
|
|
));
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* We check that something is really changed.
|
|
|
|
*/
|
|
|
|
public function clean()
|
|
|
|
{
|
|
|
|
$this->cleaned_data = parent::clean();
|
|
|
|
// As soon as we know that at least one change was done, we
|
|
|
|
// return the cleaned data and do not go further.
|
|
|
|
if (strlen(trim($this->cleaned_data['content']))) {
|
|
|
|
return $this->cleaned_data;
|
|
|
|
}
|
|
|
|
if ($this->show_full) {
|
|
|
|
$status = $this->issue->get_status();
|
|
|
|
if (trim($this->cleaned_data['status']) != $status->name) {
|
|
|
|
return $this->cleaned_data;
|
|
|
|
}
|
|
|
|
if (trim($this->issue->summary) != trim($this->cleaned_data['summary'])) {
|
|
|
|
return $this->cleaned_data;
|
|
|
|
}
|
|
|
|
$owner = self::findUser($this->cleaned_data['owner']);
|
|
|
|
if ((is_null($owner) and !is_null($this->issue->get_owner()))
|
|
|
|
or (!is_null($owner) and is_null($this->issue->get_owner()))
|
|
|
|
or ((!is_null($owner) and !is_null($this->issue->get_owner())) and $owner->id != $this->issue->get_owner()->id)) {
|
|
|
|
return $this->cleaned_data;
|
|
|
|
}
|
|
|
|
$tags = array();
|
|
|
|
for ($i=1;$i<7;$i++) {
|
|
|
|
if (strlen($this->cleaned_data['label'.$i]) > 0) {
|
|
|
|
if (strpos($this->cleaned_data['label'.$i], ':') !== false) {
|
|
|
|
list($class, $name) = explode(':', $this->cleaned_data['label'.$i], 2);
|
|
|
|
list($class, $name) = array(trim($class), trim($name));
|
|
|
|
} else {
|
|
|
|
$class = 'Other';
|
|
|
|
$name = trim($this->cleaned_data['label'.$i]);
|
|
|
|
}
|
|
|
|
$tags[] = array($class, $name);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
$oldtags = $this->issue->get_tags_list();
|
|
|
|
foreach ($tags as $tag) {
|
|
|
|
$found = false;
|
|
|
|
foreach ($oldtags as $otag) {
|
|
|
|
if ($otag->class == $tag[0] and $otag->name == $tag[1]) {
|
|
|
|
$found = true;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if (!$found) {
|
|
|
|
// new tag not found in the old tags
|
|
|
|
return $this->cleaned_data;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
foreach ($oldtags as $otag) {
|
|
|
|
$found = false;
|
|
|
|
foreach ($tags as $tag) {
|
|
|
|
if ($otag->class == $tag[0] and $otag->name == $tag[1]) {
|
|
|
|
$found = true;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if (!$found) {
|
|
|
|
// old tag not found in the new tags
|
|
|
|
return $this->cleaned_data;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
// no changes!
|
|
|
|
throw new Pluf_Form_Invalid(__('No changes were entered.'));
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Save the model in the database.
|
|
|
|
*
|
|
|
|
* @param bool Commit in the database or not. If not, the object
|
|
|
|
* is returned but not saved in the database.
|
|
|
|
* @return Object Model with data set from the form.
|
|
|
|
*/
|
|
|
|
function save($commit=true)
|
|
|
|
{
|
|
|
|
if ($this->isValid()) {
|
|
|
|
if ($this->show_full) {
|
|
|
|
// Add a tag for each label
|
|
|
|
$tags = array();
|
|
|
|
$tagids = array();
|
|
|
|
for ($i=1;$i<7;$i++) {
|
|
|
|
if (strlen($this->cleaned_data['label'.$i]) > 0) {
|
|
|
|
if (strpos($this->cleaned_data['label'.$i], ':') !== false) {
|
|
|
|
list($class, $name) = explode(':', $this->cleaned_data['label'.$i], 2);
|
|
|
|
list($class, $name) = array(trim($class), trim($name));
|
|
|
|
} else {
|
|
|
|
$class = 'Other';
|
|
|
|
$name = trim($this->cleaned_data['label'.$i]);
|
|
|
|
}
|
|
|
|
$tag = IDF_Tag::add($name, $this->project, $class);
|
|
|
|
$tags[] = $tag;
|
|
|
|
$tagids[] = $tag->id;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
// Compare between the old and the new data
|
|
|
|
$changes = array();
|
|
|
|
$oldtags = $this->issue->get_tags_list();
|
|
|
|
foreach ($tags as $tag) {
|
|
|
|
if (!Pluf_Model_InArray($tag, $oldtags)) {
|
|
|
|
if (!isset($changes['lb'])) $changes['lb'] = array();
|
|
|
|
if ($tag->class != 'Other') {
|
|
|
|
$changes['lb'][] = (string) $tag; //new tag
|
|
|
|
} else {
|
|
|
|
$changes['lb'][] = (string) $tag->name;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
foreach ($oldtags as $tag) {
|
|
|
|
if (!Pluf_Model_InArray($tag, $tags)) {
|
|
|
|
if (!isset($changes['lb'])) $changes['lb'] = array();
|
|
|
|
if ($tag->class != 'Other') {
|
|
|
|
$changes['lb'][] = '-'.(string) $tag; //new tag
|
|
|
|
} else {
|
|
|
|
$changes['lb'][] = '-'.(string) $tag->name;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
// Status, summary and owner
|
|
|
|
$status = IDF_Tag::add(trim($this->cleaned_data['status']), $this->project, 'Status');
|
|
|
|
if ($status->id != $this->issue->status) {
|
|
|
|
$changes['st'] = $status->name;
|
|
|
|
}
|
|
|
|
if (trim($this->issue->summary) != trim($this->cleaned_data['summary'])) {
|
|
|
|
$changes['su'] = trim($this->cleaned_data['summary']);
|
|
|
|
}
|
|
|
|
$owner = self::findUser($this->cleaned_data['owner']);
|
|
|
|
if ((is_null($owner) and !is_null($this->issue->get_owner()))
|
|
|
|
or (!is_null($owner) and is_null($this->issue->get_owner()))
|
|
|
|
or ((!is_null($owner) and !is_null($this->issue->get_owner())) and $owner->id != $this->issue->get_owner()->id)) {
|
|
|
|
$changes['ow'] = (is_null($owner)) ? '---' : $owner->login;
|
|
|
|
}
|
|
|
|
// Update the issue
|
|
|
|
$this->issue->batchAssoc('IDF_Tag', $tagids);
|
|
|
|
$this->issue->summary = trim($this->cleaned_data['summary']);
|
|
|
|
$this->issue->status = $status;
|
|
|
|
$this->issue->owner = $owner;
|
|
|
|
}
|
|
|
|
// Create the comment
|
|
|
|
$comment = new IDF_IssueComment();
|
|
|
|
$comment->issue = $this->issue;
|
|
|
|
$comment->content = $this->cleaned_data['content'];
|
|
|
|
$comment->submitter = $this->user;
|
|
|
|
if (!$this->show_full) $changes = array();
|
|
|
|
$comment->changes = $changes;
|
|
|
|
$comment->create();
|
|
|
|
$this->issue->update();
|
2008-08-12 10:46:09 +00:00
|
|
|
$this->issue->setAssoc($this->user); // interested user.
|
2008-07-25 08:26:05 +00:00
|
|
|
return $this->issue;
|
|
|
|
}
|
|
|
|
throw new Exception(__('Cannot save the model from an invalid form.'));
|
|
|
|
}
|
|
|
|
}
|