Milan
Account created on 1 April 2009, over 15 years ago
#

Merge Requests

Recent comments

๐Ÿ‡ฎ๐Ÿ‡นItaly Noemi Milan

Hi all, eventually I solved applying the patch from this Core issue: https://www.drupal.org/project/drupal/issues/3106205 ๐Ÿ› Length of menu_tree.url and menu_tree.route_param_key are too short (255 characters) Needs review .

The issue here is that menu route field is too short to hold the whole string generated from facets.

๐Ÿ‡ฎ๐Ÿ‡นItaly Noemi Milan

Noemi โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡นItaly Noemi Milan

I tested patch #15 ๐Ÿ“Œ Compatibility with s3fs module Needs review and it works fine on 2.0.x version.

๐Ÿ‡ฎ๐Ÿ‡นItaly Noemi Milan

We're facing the same issue, after upgrading from 1.8.0. I don't know if there's a previous version not affected.

๐Ÿ‡ฎ๐Ÿ‡นItaly Noemi Milan

@pookmish I realized the problem with the count is not directly related to this issue, but the use of an offset.
In short, the total count is equal to the count + the offset and maybe this is the expected behavior.

๐Ÿ‡ฎ๐Ÿ‡นItaly Noemi Milan

The patch is working fine with 9.5.11, but the total count is not correct, since it returns the items per page + 1.

Does it make sense to use $view->total_rows to populate the count property in meta?

๐Ÿ‡ฎ๐Ÿ‡นItaly Noemi Milan

I updated the module to the latest version (1.12.0), with all its dependencies, and the problem was fixed.
The strangest thing is that I haven't seen anything in the Drupal logs when visiting the broken page.

๐Ÿ‡ฎ๐Ÿ‡นItaly Noemi Milan

Noemi โ†’ made their first commit to this issueโ€™s fork.

Production build 0.71.5 2024