Projekt

Allgemein

Profil

« Zurück | Weiter » 

Revision 9e4a9fec

Von Moritz Bunkus vor etwa 10 Jahren hinzugefügt

  • ID 9e4a9fec6b6311be5280a7861aad014e8465a749
  • Vorgänger 77c9d23b
  • Nachfolger 875e89ee

SL::DB::BackgroundJob: refactoring von set_data für mehr programmatische Sicherheit

$self->data kann durchaus leer sein, und dann gibt YAML::Load schlicht
einen leeren String und keine Hashref zurück.

Daher $self->data_as_hash benutzen, das bereits für die meisten Fälle
gewappnet ist. Außerdem einfachere Hash-Zuweisung.

Unterschiede anzeigen:

SL/DB/BackgroundJob.pm
sub set_data {
my ($self, %data) = @_;
my $data = YAML::Load($self->data);
$data->{$_} = $data{$_} for keys %data;
$self->data(YAML::Dump($data));
$self->data(YAML::Dump({
%{ $self->data_as_hash },
%data,
}));
$self;
}

Auch abrufbar als: Unified diff