Projekt

Allgemein

Profil

« Zurück | Weiter » 

Revision f3aa6818

Von Moritz Bunkus vor mehr als 10 Jahren hinzugefügt

  • ID f3aa68189371cfcbde72a40f4a909356e14e3c94
  • Vorgänger 03f5aaa3
  • Nachfolger 013804fd

Refactoring: explizites $rspec->delete_items vor ->copy_from inzwischen unnötig

...seitdem die BEFORE-DELETE-Trigger für die Items richtig funktionieren.

Unterschiede anzeigen:

SL/DB/RequirementSpec.pm
156 156
  return $self;
157 157
}
158 158

  
159
sub delete_items {
160
  my ($self) = @_;
161

  
162
  my $worker = sub {
163
    # First convert all items to sections so that deleting won't
164
    # violate foreign key constraints on parent_id.
165
    SL::DB::Manager::RequirementSpecItem->update_all(
166
      set   => { parent_id => undef, item_type => 'section' },
167
      where => [
168
        requirement_spec_id => $self->id,
169
        '!parent_id'        => undef,
170
      ]);
171

  
172
    # Now delete all items in one go.
173
    SL::DB::Manager::RequirementSpecItem->delete_all(where => [ requirement_spec_id => $self->id ]);
174

  
175
    # Last clear values in ourself.
176
    $self->items([]);
177
  };
178

  
179
  return $self->db->in_transaction ? $worker->() : $self->db->do_transaction($worker);
180
}
181

  
182 159
sub previous_version {
183 160
  my ($self) = @_;
184 161

  

Auch abrufbar als: Unified diff