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.

Sign up for GitHub

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

Jump to bottom

FileAssetContainers not found when using case #30457

Closed
erickgonzalez opened this issue Oct 25, 2024 · 3 comments · Fixed by #30543
Closed

FileAssetContainers not found when using case #30457

erickgonzalez opened this issue Oct 25, 2024 · 3 comments · Fixed by #30543
Assignees
@dsolistorres
Labels
OKR : Customer Support Owned by Scott Priority : 3 Average QA : Approved Release : 23.10.24 v20 Included in LTS patch release 23.10.24 v20 Release : 24.04.24 v12 Included in LTS patch release 24.04.24 v12 Release : 24.11.14 Bug Fixing Team : Bug Fixers Type : Defect

Comments

@erickgonzalez
Copy link
Contributor

Parent Issue

No response

Problem Statement

In the Template Portlet, FileAssetContainers are not consistently listed when searched for with the case they were originally created in. This leads to an issue where users can only locate FileAssetContainers by using lowercase search terms, regardless of the original case. This inconsistency may result in user frustration, reduced efficiency, and potential confusion, as users might not find the assets they are looking for if they don’t use lowercase during the search.

NOTE: This only happens to FileAssetContainers; for normal containers, it finds them regardless of the case used.

Steps to Reproduce

1- Create a FileAssetContainer with the title UPPERCASE
2- Go to the Templates Portlet
3- Create a new design Template
4- Try to find the created container by typing UPPERCASE
5- Nothing is found
6- Try to find the created container by typing uppercase
7- Container is found

Screen.Recording.2024-10-25.at.10.08.22.AM.mov

Acceptance Criteria

Regardless of the case used to search the containers, they should return.

dotCMS Version

24.04 LTS
demo (24.10.16)

Proposed Objective

Customer Support

Proposed Priority

Priority 3 - Average

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.

https://dotcms.freshdesk.com/a/tickets/28191

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

If you search for them in lowercase, it finds them

Sub-Tasks & Estimates

No response
@erickgonzalez erickgonzalez moved this from New to Current Sprint Backlog in dotCMS - Product Planning Oct 28, 2024
@dsolistorres dsolistorres self-assigned this Oct 29, 2024
@dsolistorres dsolistorres moved this from Current Sprint Backlog to In Progress in dotCMS - Product Planning Oct 29, 2024
dsolistorres added a commit that referenced this issue Oct 31, 2024
@dsolistorres
fix(containers) : change search functionality to case insensitive for…
a3b66e7 
… file containers (#30457)
@dsolistorres dsolistorres mentioned this issue Oct 31, 2024
Merged
1 task
@github-actions GitHub Actions
Copy link

PRs:

dsolistorres added a commit that referenced this issue Oct 31, 2024
@dsolistorres
fix(containers) : change search functionality to case insensitive for…
2a8bb65 
… file containers (#30457)
@dsolistorres dsolistorres moved this from In Progress to In Review in dotCMS - Product Planning Oct 31, 2024
dsolistorres added a commit that referenced this issue Nov 1, 2024
@dsolistorres
fix(containers) : change search functionality to case insensitive for…
02ffec8 
… file containers (#30457)
dsolistorres added a commit that referenced this issue Nov 4, 2024
@dsolistorres
fix(containers) : change search functionality to case insensitive for…
a1a8375 
… file containers (#30457)
github-merge-queue bot pushed a commit that referenced this issue Nov 12, 2024
@dsolistorres @erickgonzalez
fix(containers) : change search functionality to case insensitive for…
b9c2da1 
… file containers (#30543)

Closes #30457

### Proposed Changes
* Modified the `findFolderAssetContainers` method to ensure container
name searches are case-insensitive

### Checklist
- [x] Tests

Co-authored-by: erickgonzalez <[email protected]>
@github-project-automation github-project-automation bot moved this from In Review to Internal QA in dotCMS - Product Planning Nov 12, 2024
@erickgonzalez
Copy link
Contributor Author

Working as it should, now you can search file asset containers regardless of the case.

Screenshot 2024-11-13 at 10 49 00 AM

Screenshot 2024-11-13 at 10 48 46 AM

Screenshot 2024-11-13 at 10 49 23 AM

@erickgonzalez erickgonzalez moved this from Internal QA to QA - Backlog in dotCMS - Product Planning Nov 13, 2024
@bryanboza
Copy link
Contributor

Fixed, now it is working as expected
Image

@erickgonzalez erickgonzalez added the LTS: Needs Backport Ticket that will be added to LTS label Nov 14, 2024
@erickgonzalez erickgonzalez added the LTS: Next Patch Shortlisted of issues that will be included in the upcoming LTS patch label Nov 28, 2024
gortiz-dotcms added a commit that referenced this issue Dec 5, 2024
@gortiz-dotcms
#30457 include in 24.04.24 LTS
9bf212a
erickgonzalez added a commit that referenced this issue Dec 5, 2024
@erickgonzalez
#30457 include in 23.10.24 LTS
87843ec
@erickgonzalez erickgonzalez added Release : 23.10.24 v20 Included in LTS patch release 23.10.24 v20 Release : 24.04.24 v12 Included in LTS patch release 24.04.24 v12 and removed LTS: Needs Backport Ticket that will be added to LTS LTS: Next Patch Shortlisted of issues that will be included in the upcoming LTS patch labels Dec 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dsolistorres dsolistorres

Labels
OKR : Customer Support Owned by Scott Priority : 3 Average QA : Approved Release : 23.10.24 v20 Included in LTS patch release 23.10.24 v20 Release : 24.04.24 v12 Included in LTS patch release 24.04.24 v12 Release : 24.11.14 Bug Fixing Team : Bug Fixers Type : Defect
Projects
dotCMS - Product Planning
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(containers) : change search functionality to case insensitive for file containers dotCMS/core
3 participants
@bryanboza @dsolistorres @erickgonzalez