Opened 10 years ago

Closed 10 years ago

#9288 closed Bug Report (Fixed)

Change 24895- the toLower is the wrong place

Reported by: cpw@… Owned by: beirdo
Priority: trivial Milestone: 0.24.1
Component: Plugin - MythGallery Version: 0.24
Severity: medium Keywords:
Cc: Ticket locked: no


Hi Myth 0.24 broke mythgallery video playing for me. All my files are (unfortunately) in upper case: *.MOV mostly. The Changeset 24895 adding in mp4 and cleaning up the upper/lower stuff in the filter does one thing slightly wrong. The change does this:

        if ((*it).toLower().contains(fi.suffix())) 

However, you actually meant to toLower the file suffix, so it matches the filter contents:

        if ((*it).contains(fi.suffix().toLower()))

I have compiled this change locally and confirm it works.


Change History (3)

comment:1 Changed 10 years ago by beirdo

Milestone: unknown0.24.1
Priority: majortrivial
Status: newassigned

comment:2 Changed 10 years ago by beirdo

Committed in 641d6e65 for master, fe9d48e5 for fixes/0.24

comment:3 Changed 10 years ago by beirdo

Resolution: Fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.