| Re: XoopsLocal functions are exsit for using! |
| by irmtfan on 2012/12/9 3:16:17 You misunderstood me Trabis. I meant a new function in XoopsLocal class for input time. It is just a new name for better declaration. for example last core members defined a new function for output time XoopsLocal:formatTimestamp but we could used php date function XoopsLocal:date Anyway, we could have XoopsLocal:date_strtotime or XoopsLocal:strtotime and it is up to core developers to choose. I dont have any problem with that. Thank you for your works. I see your recent commits and I know there is a lot of works remained. |
| Re: XoopsLocal functions are exsit for using! |
| by trabis on 2012/12/8 14:21:35 There is no way we can force a developer to use the core strtotime so, it is irrelevant to use it as function or as class method for that propose. I prefer to use XoopsLocal, it is the right place for it. If we do a function, it will just mirror to this class. That joomla function has nothing to do with localization. It just helps developers with type checking. Btw, the new sf tracker looks great! |
| Re: XoopsLocal functions are exsit for using! |
| by irmtfan on 2012/12/8 8:41:53 very good. You knows better how to implement features in the next core. I think i sent all needed features for localization and customization to sf.net so finding them is easy for developers like you. the strtotime feature request is this: https://sourceforge.net/p/xoops/feature-requests/381/ as you can see Joomla used a new function. le="color: #000000"><?php function date_strtotime($date) { $strtotime = strtotime($date); if ($strtotime === -1 || $strtotime === false) { return false; } else { return $strtotime; } } maybe it is better to have a new function for input time. |
| Re: XoopsLocal functions are exsit for using! |
| by trabis on 2012/12/7 0:58:33 Quote:
I've been absent for a while and did not have the time to go through the bugs/features on sourceforge. I've read the proposal and I liked it very much. I'll try to implement the proposed features as soon as possible. |
| Re: XoopsLocal functions are exsit for using! |
| by irmtfan on 2012/12/6 6:16:28 Thank you so much for guiding me to magic call i did not aware of this method. But the point of this topic is how we can force/guide developers to use XoopsLocal class. As for a list of requierd php functions i finished my proposal for "xoops 2.6 localization" months ago and i dont have anything to add. For localizing strtotime php function i also sent a feature request. IMO defining a xoops function for converting date to int is better method. (like Joomla) |