Small fix to run this on Windows (Invalid CRT Parameters error)

trjp

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

Bleistivt

You might want to add that here, so it gets fixed in one go:

https://github.com/vanilla/vanilla/issues/2027

x00

Nice, but you don't report bugs here

open an issue

https://github.com/vanilla/vanilla/issues

hgtonight

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.