Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed Search in activity report for full name in relation #11651

Merged
merged 3 commits into from
Aug 22, 2022

Conversation

inietov
Copy link
Collaborator

@inietov inietov commented Aug 11, 2022

Description

Well this is not a 'cool' or 'good looking', probably not 'the better'. In fact, I'm pretty sure it's not the better. But yeah, it's 'a fix'. When searching trough the Activity Report if you know the complete name of the user, the search doesn't return any result.

I added a condition to do this in the searchable trait when looking inside the relations of the action_logs table. I spent more hours than I like to admit looking for a solution and this is the only thing I could think of. Let me know if something is escaping my mind, and if not, well, I think this is working for now.

Fixes internal freshdesk 29988

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Test Configuration:

  • PHP version: 8.1
  • MySQL version: 8.0.23
  • Webserver version: nginx/1.19.8
  • OS version: Debian 10

Checklist:

@snipe
Copy link
Owner

snipe commented Aug 22, 2022

This is legitimately pretty terrible lol - but I have no better suggestions right now that are not somehow worse. I'm sure there's a more elegant way down the road, but it will take some time to figure out. Thanks for this :)

@snipe snipe merged commit 3ac84ea into snipe:develop Aug 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants