Account created on 16 February 2010, almost 15 years ago
  • Systems Architect at CI&TΒ 
#

Merge Requests

Recent comments

πŸ‡§πŸ‡·Brazil rodrigoeg

Closing since this has been fixed already on another task

πŸ‡§πŸ‡·Brazil rodrigoeg

Branch 2.x already works with Drupal 10, so closing this task

πŸ‡§πŸ‡·Brazil rodrigoeg

There is already the branch 2.0.x compatible with Drupal 10, so changing to this one instead.
There was already a ticket make it compatible with Drupal 10 #3286732

πŸ‡§πŸ‡·Brazil rodrigoeg

Closing since most people seen that have their problem resolved

πŸ‡§πŸ‡·Brazil rodrigoeg

As an update, this site mentioned does not use Store Locator module anymore, so it is not possible to check how it was done

πŸ‡§πŸ‡·Brazil rodrigoeg

Closing since no reply was added, so considering this as solved

πŸ‡§πŸ‡·Brazil rodrigoeg

Probably it requires some changes to render the link correctly, so moving to a bug report

πŸ‡§πŸ‡·Brazil rodrigoeg

There might be other ways of overwriting the details via Store Locator Javascript API. Copying the javascript content might be causing other problems.
As this ticket is very old, I am not sure if you are still facing this challenge.

πŸ‡§πŸ‡·Brazil rodrigoeg

I was not able to simulate this bug. I am testing on Drupal 10.
I am using on a clean installation. If anyone has the steps of the configuration needed, could you please share more information?

πŸ‡§πŸ‡·Brazil rodrigoeg

Hello,

This issue was most probably fixed by πŸ› Problems when some module uses node_view for index nodes Fixed . I have tested with version 2.x and this bug is not there, or maybe I am testing using incorrect steps.

πŸ‡§πŸ‡·Brazil rodrigoeg

@Quentin Massez, please check if the solution of πŸ› File downloads restricted in disabled language depending on browser language Fixed fixes your issues. I am not sure if private files should be controlled by another way.

πŸ‡§πŸ‡·Brazil rodrigoeg

@Anybody
I believe the fix πŸ› Problems when some module uses node_view for index nodes Fixed applied on 2.x will also solve this problem, since now there is a way to disable verification a list of routes, as mentioned in this comment https://www.drupal.org/project/content_language_access/issues/2672542#co... πŸ› Problems when some module uses node_view for index nodes Fixed . You will be able to include "system.files" in the list of routes in the configuration page.

πŸ‡§πŸ‡·Brazil rodrigoeg

Closing since I was not able to reproduce I didn't receive more details about the issue.

And version D9/D10 is on branch 2.x

πŸ‡§πŸ‡·Brazil rodrigoeg

Thanks!
Fixes applied on branch 2.x

πŸ‡§πŸ‡·Brazil rodrigoeg

Automated tests have been fixed here: https://git.drupalcode.org/project/content_language_access/-/pipelines/1...

New release will be created soon with other adjustments as well (2.x)

πŸ‡§πŸ‡·Brazil rodrigoeg

Thanks, all!
Merged into version 2.x

πŸ‡§πŸ‡·Brazil rodrigoeg

rodrigoeg β†’ made their first commit to this issue’s fork.

πŸ‡§πŸ‡·Brazil rodrigoeg

Thanks, all!
Merged on branch 2.x (with a few adjustments: moved hook_update to hook_install, since the previous patch has never been pushed to the repository).

Removed that tag to backport to Drupal 7 because of low usage. In case someone needs backport, please submit a separate issue

πŸ‡§πŸ‡·Brazil rodrigoeg

Code merged into 8.x-1.x branch, but noticed that automated testing are not detected on D10, so more adjustments might need to be done to fix this

πŸ‡§πŸ‡·Brazil rodrigoeg

rodrigoeg β†’ made their first commit to this issue’s fork.

Production build 0.71.5 2024