[dokuwiki] New PR: Change `const` use to `var` for Safari 9 (on iOS)

  • From: "bwanders" <wiki@xxxxxxxxxxxx>
  • To: dokuwiki@xxxxxxxxxxxxx
  • Date: Tue, 19 Jun 2018 15:18:45 +0200 (CEST)

Hi,

bwanders opened a new pull request at 
https://github.com/splitbrain/dokuwiki/pull/2430:

Safari 9 in iOS does not support the use of `const` in strict mode. 
Unfortunately, some of us are stuck with Safari 9 due to not being able to 
upgrade.

I am a big fan of progressive enhancement, but because all javascript code is 
packed together and sent off to the client, this small incompatibility breaks 
all javascript for safari 9 browsers. Switching these two keywords makes 
everything work again.

There are no other uses of `const` in the codebase as far as I can find. This 
change has little to no impact on anything else, but slighlty extends the range 
of supported browsers.

(I could not find a specific listing of supported browsers on 
https://www.dokuwiki.org/browser. While I agree that Safari 9 is not exactly ;
"an up-to-date version", it seems to me that this was an accidental 
incompatibility.)

Please help us to review this pull request, so new contributors get feedback in 
a timely manner.

4afb24b0-73c3-11e8-90d3-fa2ec96e9f77


-- 
DokuWiki mailing list - more info at
http://www.dokuwiki.org/mailinglist

Other related posts:

  • » [dokuwiki] New PR: Change `const` use to `var` for Safari 9 (on iOS) - bwanders