Browser caching

[simplemachines]

Browser caching

[Spuds]

Perhaps that should have made that a variable so we do not have to edit all those files everytime

[DrDeejay]

@Spuds I can do that, if you think it's fine :)

[DrDeejay]

Done:
https://github.com/DrDeejay/playpen/commit/91623e9d60deb1f3d8378a85efd51b366cbc876b
and
https://github.com/DrDeejay/playpen/commit/658d5d895fb251f081ad2e42fec6bc939bc82beb

I'm sure I've missed many things, but I wanted to be sure that it's fine this way before I start editing the other cache versions. :)

[norv]

There are many other places where ?fin20 is used, not only index template.

Also, I don't think yet another global like this is an answer, we're trying to remove them globals, not add to them.

[DrDeejay]

I understand. :) But a question though, in which file should it be put? I was thinking of using the context variable, but that one is defined later in index.php so I didn't know if it would be fine there. What do you think?

[Spuds]

My gut reaction was not to use a global (well once I saw it of course).

There are well over 40 places that ?fin20 is used, which is why they are missed from time to time :P

Don't know how folks feel about defining a constant for this purpose define('SMF_BROWSER_SALT', '?dev21') or some such thing. Don't know really.

[DrDeejay]

A constant sounds good to me :)

[emanuele45]

str_replace(' ', '', $context['forum_version']);

:P

[Spuds]

I did consider that, but thought it was a bit clunky, even fro 2,1 :P

[Spuds]

Looks like we are going to leave this old school for this round ... find/replace