Wedge

Public area => Bug reports => The Pub => Archived fixes => Topic started by: Arantor on February 20th, 2013, 12:13 AM

Title: Serious error - current theme
Post by: Arantor on February 20th, 2013, 12:13 AM

For some reason when Current Theme is triggered, and only Current Theme (not the entire Themes and Layout section, it's only the theme settings for an actual theme), something is broken whereby it proceeds to dump the entirety of all language strings, vomiting 95 errors into the log.

I'll look into this one tonight as part of my language changes in general.

Title: Re: Serious error - current theme
Post by: Arantor on February 20th, 2013, 01:28 AM

Yay for my language caching changes!

$txt is emptied out and when the language file isn't found, it wasn't being put back. Now that sounds fine and all until we get to Current Theme which implicitly asks for ThemeStrings - which doesn't exist.

Fixed, will commit next commit.

Title: Re: Serious error - current theme
Post by: Nao on February 21st, 2013, 11:18 PM

I never even noticed that bug..?!

Title: Re: Serious error - current theme
Post by: Arantor on February 21st, 2013, 11:21 PM

You wouldn't have done, I only caused it with my caching changes, and I fixed it the same night. ;)

Title: Re: Serious error - current theme
Post by: Nao on February 21st, 2013, 11:29 PM

Quote from Arantor on February 21st, 2013, 11:21 PM

You wouldn't have done, I only caused it with my caching changes, and I fixed it the same night. ;)

Oh, all right...

Okay, bed time for me. I see you just made another commit that changes language files... I'm a bit wary of language files these days... :^^;:

Title: Re: Serious error - current theme
Post by: Arantor on February 21st, 2013, 11:32 PM

Eh, the changes were tiny. I even did the French translation for you on one of two files changes. ;)