We had the same issue and the patch fixed the problem. As noted in #5, the str_getcsv() has an issue with "," in an address.
Huge respect for finding the root of the problem! We got stuck at the same problem. The patch from #56 seems to solve the problem in a very nice way. This really should be pushed forward to be able to get into dev.
BTW: Cookiebot should not rely on third-party libraries being available on the site. This seems like a huge design problem.
The comment from #4 worked as expected. For Google Tag Module: It also works if you enable only "Anonymous Users" only tracking
This seems to be an issue with the Gin theme. We have the same problem and switching to a core theme works as expected. I was not yet able to find the commit/issue that triggered the issue.
DrColossos β changed the visibility of the branch 2.1.x to active.
DrColossos β changed the visibility of the branch 2.1.x to hidden.
DrColossos β created an issue.
I started working on this feature. The behavior is now the same as before but <a>
and <img>
get ignored. There are quite a few loops inside the function but I don't think that this affects performance.
DrColossos β made their first commit to this issueβs fork.
Updated for 6.1.x
We had the same problem in our dev environment. The issue was, that the original image was not available. After stage_file_proxy downloaded it properly, the cropping did work as expected.
I re-added the console.log to make merging with the D10 branch easier.
There are many more D10 issues. The Bot only finds a few. I opened a merge request that does all the steps:
* version constraints
* replace render()
with the proper service
* add accessCheck()
to all queries
* replace jqurey.once
with core
version
* replace obsolete jquery.ui.*
libraries with the contrib ones
* added composer.json
to properly install these contrib modules
* cleaned up a few console.log()
statements
DrColossos β made their first commit to this issueβs fork.
Merged, thank you
I created a fork with the merged changes if anybody needs to use it with composer.
DrColossos β made their first commit to this issueβs fork.
DrColossos β created an issue.
This does not yet match across fields
I added a config that allows to match only once. This does not take upper/lower casing into account. Additionally, I unified the "matches
" array it results in nested <span class="glos-term"...
because the same value could be replaced mutliple times
DrColossos β created an issue.
DrColossos β made their first commit to this issueβs fork.
#172 applies and works in 9.5.3