[dokuwiki] New PR: getGoogleQuery: ignore if query includes a full URL

  • From: "phy25" <wiki@xxxxxxxxxxxx>
  • To: dokuwiki@xxxxxxxxxxxxx
  • Date: Mon, 21 Oct 2019 01:42:13 +0200 (CEST)

Hi,

phy25 opened a new pull request at 
https://github.com/splitbrain/dokuwiki/pull/2891:

We don't want to split the URL to highlight the "query", especially when q is 
the URL of the page itself - e.g. Google Form's redirect 
https://www.google.com/url

This will also ignore queries like `syntax site:https://www.dokuwiki.org` but ;
it should be fine. Just don't want to use a full parser here.

This fixes #2848. I also added tests and removed a legacy workaround for PHP 
5.3 in the function.

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

3cf5e200-f393-11e9-8082-3194e5dfaa18


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

Other related posts:

  • » [dokuwiki] New PR: getGoogleQuery: ignore if query includes a full URL - phy25