|
From: Nathan V. <na...@th...> - 2000-12-30 02:48:26
|
On Fri, 29 Dec 2000, Brian Aker wrote:
> I could just be having a slow day today, but I am not
> sure what your patch is for.
The current code is broken; if you try to save the colors they all go
blank. Here are a couple of elaborating comments though:
> This is what is in the current code:
> if ($form->{colorpreview}) {
> $colorblock_clean = $colorblock =
> join ',', @{$form}{qw[fg0 fg1 fg2 fg3 bg0 bg1 bg2 bg3]};
I moved this line above the if statement.
>
> # the #s will break the url
> $colorblock_clean =~ s/#//g;
>
> } else {
> $colorblock = $slashdb->getBlock($form->{color_block}, 'block');
> }
I took out this else statement; $colorblock should get loaded from the
form (as in the line above which I moved), not from the database...
My afternoon is rapidly scattering too...
-n
|