damienhaynes / moving-pictures

Moving Pictures is a movies plug-in for the MediaPortal media center application. The goal of the plug-in is to create a very focused and refined experience that requires minimal user interaction. The plug-in emphasizes usability and ease of use in managing a movie collection consisting of ripped DVDs, and movies reencoded in common video formats supported by MediaPortal.
12 stars 6 forks source link

Decimals not working in filters #1068

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
OK, i'm trying to set up a filter for movies with a score from 8 till 9, but 
since there isn't an option for => (greater or equal) i was using >7.9 and <9, 
but when i hit 'OK' and reopen the filter config, the dot or comma gets 
removed. So the filter is set to >79 and <9 which (off course) gives 0 results 
back.

What is the expected output? What do you see instead?
Results from movies with a score >7.9 and <9

What version of Moving Pictures are you using?
1.3.0.1337

What version of MediaPortal?
1.2.1

What Skin?
StreamedMP

The solution could also be a 'greater or equal' and 'smaller or equal' rule, or 
the decimals should be respected.

Original issue reported on code.google.com by jeroen.s...@gmail.com on 28 Nov 2011 at 8:47

GoogleCodeExporter commented 9 years ago

Original comment by conrad.john on 28 Nov 2011 at 8:48

GoogleCodeExporter commented 9 years ago

Original comment by conrad.john on 30 Dec 2011 at 6:50

GoogleCodeExporter commented 9 years ago
this has been fixed, I think?
I'm able to do "greater than" 7 and "less than" 10... 

Close?

Original comment by michel.zehnder on 4 Feb 2012 at 2:16

GoogleCodeExporter commented 9 years ago
Sorry, I mean it works to:  >7.9  & <8.1, saves it correctly, loads it 
correctly.

Original comment by michel.zehnder on 4 Feb 2012 at 2:36

GoogleCodeExporter commented 9 years ago
It hasn't been fixed and it doesn't work.

Original comment by jeroen.s...@gmail.com on 4 Feb 2012 at 2:47

GoogleCodeExporter commented 9 years ago
jeroen, are using a regional setting that does not use decimals but maybe 
commas instead?

Original comment by damien.haynes@gmail.com on 5 Feb 2012 at 1:39

GoogleCodeExporter commented 9 years ago
The issue is still present for people in certain regions. Just saying I am 
aware of the problem and will try to get it fixed. And trying to prevent "does 
it or doesn't it work" dialog. :P

And of course sorry for the problem guys. ><

Original comment by conrad.john on 5 Feb 2012 at 3:31

GoogleCodeExporter commented 9 years ago
Bit short on the previous reply, but tested it with v1.3.0.1351 and issue is 
indeed still present for me.

@Damien: Yes, Dutch so commas instead of dots for me.

@John: OK :) 

Original comment by jeroen.s...@gmail.com on 5 Feb 2012 at 9:26

GoogleCodeExporter commented 9 years ago
Issue 1086 has been merged into this issue.

Original comment by conrad.john on 6 Mar 2012 at 8:03

GoogleCodeExporter commented 9 years ago
THis should eb resolved now, can anyone give this a try?

Test build:
http://moving-pictures.tv/files/moving-pictures-1.4.0.1373-alpha.exe

Original comment by conrad.john on 6 Mar 2012 at 8:04

GoogleCodeExporter commented 9 years ago
I gave it a quick test John, and seems that the filter is working on first 
setup. When i re-open the Filter Editor, the . is removed again, which then 
leads to 0 results for this filter (Score greater than 79 instead of 7.9). So i 
you could make it store the . in the Filter Editor, it would be perfect.

Original comment by jeroen.s...@gmail.com on 7 Mar 2012 at 6:46

GoogleCodeExporter commented 9 years ago

Original comment by damien.haynes@gmail.com on 12 Jan 2014 at 8:25