Croatia, Montenegro
Account created on 27 April 2019, about 5 years ago
#

Merge Requests

More

Recent comments

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

@Anybody

Because of that, I disabled that option by default. If someone needs to delete logs for deleted entities, then they have options to do that.

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

I've created logic where 100 logs are loaded on cron. It checks if entities exist, and if they don't, it deletes the log. Deletion is disabled by default, but can be enabled in settings.

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

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

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

Hi, I added a new field called 'bundle,' which contains the entity type. Now, it can be filtered in the view by this 'bundle' field. This way, all fields under one entity type will be obtained together.

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

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

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro
πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

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

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

Hi @levmyshkin

The Entity log is used for tracking field changes on entities.

It's not used for deleting, so it seems like your custom integration for logging entity deletions caused an issue.

I tried the case you described, but I did not encounter any issues because the module does not log any deletions.

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

Hi @mdolnik,

This doesn't seem like a bug to me, it looks more like a custom solution you want to implement on your project.

I'm not clear on why you're calling the setEntityLoggedId() method. As you mentioned, if you want to modify logged values, you can extend the EntityLogService class and customize the logFields() method to suit your needs.

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro
πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

I can confirm that I was able to reproduce the issue on Drupal 9 as reported. Upon the first page refresh, there are no flags, but when navigating to another page, AJAX is used and everything works.

After updating to Drupal Core 10, it no longer works, not even with AJAX - nothing appears at all.

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

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

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

Hi @roman_pro

This is something that needs to be custom-made for a specific case, it's not something that should be done at the module level.

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

MilosR β†’ changed the visibility of the branch 3137304-error-when-faceting to active.

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

MilosR β†’ changed the visibility of the branch 3137304-error-when-faceting to hidden.

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

I am currently experiencing this issue on Drupal core 10.2.5 and Stage File Proxy Version 2.1.4.

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

@Asterovim

Hi, you can, for example, add a "changed" field in the view.

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

@thechanceg

Yes, it is possible, you just need to add field under General. Add the flag you need, and then in the view, add an indexed field like "User flag (indexed field)" as necessary. Additionally, you may need to add a User relationship in the view, depending on what is required.

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

@jukka792

Can you provide steps to reproduce this issue?

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro
πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

@Youcanlearnit

You're not right. Flags related to users can be indexed.

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

@jukka792

Can you check when hovering over the flag link if there's something like this in the link: "/typeerror-null-given"?

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

Thanks @BenStallings

In my case, I had a problem with facets in my custom code, which was called like this:

$request->query->get('f')

So, I changed it to this, and it works:

$request->query->all('f')

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

MilosR β†’ created an issue.

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

@larowlan which part of the code?

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

The flags created and used with the flag_search_api module do not have labels. In this case, the 'List item label' should not exist in the form. The fix is to uncheck the 'List item label,' and here is the patch that removes the 'List item label' checkbox for all flags that do not have that functionality. Therefore, in future cases, there will not be any options to check 'List item label'.

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

Hi, @jukka792, I just tested this case in a fresh new installation of Drupal Core (version 9.5.11), and indexing works as expected in both cases, whether an action is flagged or unflagged. I do not see that the issue you mentioned is caused by this module. Could you please investigate further on a fresh Drupal installation and provide more data?

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

The patch are not applying on the latest version of the module, we need to work on this.

πŸ‡­πŸ‡·Croatia MilosR Croatia, Montenegro

#91 Patch file does not apply to the latest version.

Production build 0.69.0 2024