[Bug 151357] New: Ratings can exceed 5 stars

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=151357         
           Summary: Ratings can exceed 5 stars
           Product: digikam
           Version: unspecified
          Platform: unspecified
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: general
        AssignedTo: digikam-devel kde org
        ReportedBy: maw48 cantab net


Version:           0.9.2-final (using KDE 3.5.8, Kubuntu (gutsy) 4:3.5.8-0ubuntu3.1)
Compiler:          Target: i486-linux-gnu
OS:                Linux (i686) release 2.6.22-14-386

Ratings for a picture are able to exceed 5 stars, which doesn't appear intentional!

To reproduce: click and hold within the stars on the rating "meter" in the comments / tags sidebar.  Then drag right to increase the rating.  Drag beyond the limit of the rating widget and then release.  Apply this and the picture will display a rating greater than 5 stars.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Arnd Baecker]

Hi Mark,

thanks for the report! I can reproduce and it seems simple
to fix, compile is running...
(note: B.K.O web-interface  seems broken, so I don't know
when I can attach the fix - if the fix works ;-)

Best, Arnd
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=151357         
caulier.gilles gmail com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
          Component|general                     |Albums GUI
         Resolution|                            |FIXED



------- Additional Comments From arnd.baecker web de  2007-10-26 07:50 -------
Hi Mark,

thanks for the report! I can reproduce and it seems simple
to fix, compile is running...
(note: B.K.O web-interface  seems broken, so I don't know
when I can attach the fix - if the fix works ;-)

Best, Arnd


------- Additional Comments From arnd.baecker web de  2007-10-26 08:08 -------
Created an attachment (id=21914)
 --> (http://bugs.kde.org/attachment.cgi?id=21914&action=view)
ensure rating<=5 when dragging with the mouse

In RatingWidget::mouseMoveEvent it was possible to generate
values >=5 for the position of the mouse (values < 0 are not possible
as the mouse position is always >=0). This
value was directly used as d->rating.
The attached patch ensures that pos<=RatingMax.

------- Additional Comments From arnd.baecker web de  2007-10-26 18:52 -------
SVN commit 729686 by abaecker:

Ensure that rating cannot be larger than RatingMax when dragging the mouse.

CCBUGS: 151357
TODO:KDE4PORT



 M  +2 -1      NEWS  
 M  +2 -0      digikam/ratingwidget.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=729686


------- Additional Comments From arnd.baecker web de  2007-10-26 19:15 -------
SVN commit 729699 by abaecker:

Ensure that rating cannot be smaller than RatingMin when dragging the mouse.

CCBUGS: 151357
TODO:KDE4PORT



 M  +2 -0      ratingwidget.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=729699


------- Additional Comments From caulier.gilles gmail com  2007-10-30 08:23 -------
patch backported to KDE4. I close this file now.

Gilles Caulier
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=151357         




------- Additional Comments From maw48 cantab net  2007-10-30 14:23 -------
Hi!

Thanks, I'm glad I could help.  Don't worry about getting the patch online on my account; I'm happy to know it's fixed.  It's not really causing me much difficulty (although finding 7 star rating pictures is a bit surprising!), I'm just glad my sloppy mouse work led me to find something useful ;-)

Digikam is awesome!

Cheers,
Mark
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel