Hello there!

It's just the second time I'm posting here on drupal.org, so if anything is wrong on this issue, please feel free to tell me so I can update it.

Well, regarding the issue, I found myself recently in a situation in which the acceptance of the cookies by the user was provoking some weird bugs on a drupal site. Luckily, reloading after accepting solved those bugs (or, at least, most of them). Thing is, I didn't want all the pages to reload, since the bugs were only on some specific ones and also cause I find quite anoying (for me myself and for the users) that the page reloads after the agreement.

So, to solve this, I thought that it could be great that the module could allow you to specify in which pages or routes you wanted the pages to reload after the agreement. As it could be handy, I also thought that an option to exclude, instead of include, those routes for reloading would be great.

As a result, I implemented this small patch that it has worked for me so far. I'm sure there's probably a better way to do all this, but since I'm a Drupal noob (I've been working with it for a year or so) I did it the best way I knew.

I'm attachin here the patch so anyone that may have the same problem as I can use it, and of course for it to be improved by you Drupal gurus. And well, if the module maintainers find it interesting, for it to be added to the next release, adjusting it to the lastest version.

Comments

gascon created an issue. See original summary.

gascon’s picture

Status: Needs review » Needs work

The last submitted patch, 2: eu_cookie_compliance-reload_page_extended_functionality-3001123-2-D8.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

svenryen’s picture

Hi gascon. Thanks for the patch. Would you mind looking over the coding style feedback from the testbot? Also, we need a patch for Drupal 7.

svenryen’s picture

I fixed a few code standard issues and tested the patch. Works like it should, a bit unfortunate that it doesn't support url aliases. I added a note about that in the field description.

  • svenryen authored 889ab70 on 8.x-1.x
    Issue #3001123 by gascon, svenryen: Include/Exclude routes for reloading...
svenryen’s picture

Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.