WebGUI
      Click here to register.
      
Sprechen Sie WebGUI? Parlez vous WebGUI? Se habla WebGUI? Spreekt u WebGUI?

Do you speak WebGUI? Please help us translate WebGUI into your language.



     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 88
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 [
|
]