WebGUI
      Click here to register.
      
irc://irc.freenode.net#webgui

iPhoneGuy: WebGUI is a pile of crap.
rizen: If WebGUI is such a pile of crap, why do you use it?
iPhoneGuy: Because it's the best pile of crap out there.

If this is what people who hate us are saying, imagine what people who love us will say. Come join us on IRC.


     Report a Bug > WebGUI Bug Tracker

DateTime.pm not using locale info from Calendar.pm asset (fixed)

User zylo
Date 4/7/2008 1:07 pm
Severity Minor (annoying, but not harmful)
Version WebGUI Stable 7.4.31
Views 144
Rating 1    Rate [
|
]
Karma Rank 0.000000
Previous · Next
User Message
zylo

Localization in the Calendar asset is broken, due to an error in WebGUI::DateTime.

The localization string is not used in certain cases, within the DateTime object.


I have solved the problem myself, but i am unsure how i post the patch/change.

in the new subrutine the following has been changed in the last else of the "if" construction. The bold part has been added:  

else
        {
                $self   = $class->SUPER::new(
                                (_splitMysql($_[0])),
                                time_zone       => "UTC",
                                locale=>$locale,
                                );
        }

 

Do i get points for this ;D

ps. i have attached the whole of the DateTime.pm file, with changes
applied

edit: i have attached the patch also

Best regards
Zylo

 



Attached Files
Back to Top
Rate [
|
]
 
 
Graham

Thank you for the patch.  For future reference, it is usually preferable to use the -u switch when running diff.  This generates the patch in a better format.

Fixed for 7.4.32, 7.5.10.



Back to Top
Rate [
|
]