Photo of Mark's face, taken in standard office fluorescent.Mark Smith ([staff profile] mark) wrote in [site community profile] changelog,
@ 2009-06-12 08:49 pm UTC
  • Previous Entry
  • Add to Memories
  • Tell someone about this!
  • Next Entry

[commit: http://hg.dwscoalition.org/dw-free/rev/bbe1e7bc106f]

http://bugs.dwscoalition.org/show_bug.cgi?id=880

Fix error when layout layer is deleted but the style still is defined.

Patch by [personal profile] afuna.

Files modified:
  • cgi-bin/LJ/Customize.pm
--------------------------------------------------------------------------------
diff -r 64f05824a80e -r bbe1e7bc106f cgi-bin/LJ/Customize.pm
--- a/cgi-bin/LJ/Customize.pm	Fri Jun 12 20:46:06 2009 +0000
+++ b/cgi-bin/LJ/Customize.pm	Fri Jun 12 20:49:28 2009 +0000
@@ -87,6 +87,14 @@ sub verify_and_load_style {
     my $u = shift;
 
     my $style = LJ::S2::load_style($u->prop('s2_style'));
+
+    unless ( $style && $style->{layer}->{layout} ) {
+        # we have no layout layer for this style, which causes errors in
+        # the customization interface
+        # undef current style and force them to use the site defaults
+        $u->set_prop( s2_style => 0 );
+        $style = undef;
+    }
 
     unless ($style && $style->{'userid'} == $u->{'userid'}) {
         my $theme;
--------------------------------------------------------------------------------


(1 comment) - (Post a new comment)
(Flat) (Top-level comments only)

"Cesy" - An old-fashioned quill and ink
Re: Bug 880
[personal profile] cesy
2009-06-13 11:06 am UTC (link)
Is this fixing that known error we keep getting Support requests about? If so, yay!

(Reply to this



(1 comment) - (Post a new comment)
(Flat) (Top-level comments only)