HackerOne users: Testing against this community violates our program's Terms of Service and will result in your bounty being denied.
Small fix to run this on Windows (Invalid CRT Parameters error)
I realise it's probably not something people do for production perhaps, but it's handy for testing and there's a partial fix in there already anyway ;0
In class.format.php there is a fix for the strftime issues on Windows - lines 532-536
Problem is, that only fixes $Format - it also needs to fix $FullFormat - e.g.
if (strpos($FullFormat, '%l') !== false) $FullFormat = str_replace('%l', ltrim(strftime('%I', $Timestamp), '0'), $FullFormat); if (strpos($FullFormat, '%e') !== false) $FullFormat = str_replace('%e', ltrim(strftime('%d', $Timestamp), '0'), $FullFormat);
Needs to go before current line 540
Hope that helps someone
Tagged:
0
Comments
You might want to add that here, so it gets fixed in one go:
https://github.com/vanilla/vanilla/issues/2027
My themes: pure | minusbaseline - My plugins: CSSedit | HTMLedit | InfiniteScroll | BirthdayModule | [all] - PM me about customizations
VanillaSkins.com - Plugins, Themes and Graphics for Vanillaforums OS
Nice, but you don't report bugs here
open an issue
https://github.com/vanilla/vanilla/issues
grep is your friend.
I had a "windows compatibility" locale list that changes the date formats. I lost it in the move, but you don't need to change core to support IIS/Windows. Just change the formats via a custom locale.
Search first
Check out the Documentation! We are always looking for new content and pull requests.
Click on insightful, awesome, and funny reactions to thank community volunteers for their valuable posts.