[Bug 176086] New: Recursive slidewshow only shows last Album

[yann LE HIR-2]

http://bugs.kde.org/show_bug.cgi?id=176086

           Summary: Recursive slidewshow only shows last Album
           Product: digikam
           Version: unspecified
          Platform: Gentoo Packages
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: general
        AssignedTo: [hidden email]
        ReportedBy: [hidden email]


Version:           SVN-888882 (using KDE 4.1.3)
Compiler:          cmake version 2.6-patch 2, gcc 4.1.2 (Gentoo 4.1.2 p1.0.2)
OS:                Linux
Installed from:    Gentoo Packages

I was just hacking around trying to add a new slideshow system and I find out
that asking digikam for a recursive slideshow only shows the last album.


--
Configure bugmail: http://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[yann LE HIR-2]

http://bugs.kde.org/show_bug.cgi?id=176086





--- Comment #1 from yann LE HIR <yann lehir gmail com>  2008-11-25 21:23:45 ---
Created an attachment (id=28821)
 --> (http://bugs.kde.org/attachment.cgi?id=28821)
Patch file that solves the bug.


--
Configure bugmail: http://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

http://bugs.kde.org/show_bug.cgi?id=176086


Andi Clemens andi clemens gmx net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1




--- Comment #2 from Andi Clemens <andi clemens gmx net>  2008-11-25 21:59:09 ---
There seem to be more problems. I have applied the patch (but with a foreach
loop) and started the slideshow for a folder without subfolders.
Using All (F9) loads all images into the slideshow, but using "With Subfolders"
does not load all images.
Same for a folder with subfolders. Recursive view mode displays more images
then the slideshow loads.

Gilles, Marcel,

any idea?

Andi


--
Configure bugmail: http://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

http://bugs.kde.org/show_bug.cgi?id=176086


Andi Clemens andi clemens gmx net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|general                     |SlideShow
            Version|unspecified                 |0.10.0-svn




--
Configure bugmail: http://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

http://bugs.kde.org/show_bug.cgi?id=176086





--- Comment #3 from Andi Clemens <andi clemens gmx net>  2008-11-25 22:04:05 ---
Created an attachment (id=28822)
 --> (http://bugs.kde.org/attachment.cgi?id=28822)
patch#2

slighty different patch (like in KDE3 branch)


--
Configure bugmail: http://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

http://bugs.kde.org/show_bug.cgi?id=176086


Andi Clemens andi clemens gmx net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Recursive slidewshow only   |Recursive slideshow only
                   |shows last Album            |shows last album




--
Configure bugmail: http://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[yann LE HIR-2]

http://bugs.kde.org/show_bug.cgi?id=176086





--- Comment #4 from yann LE HIR <yann lehir gmail com>  2008-11-25 22:46:17 ---
I didn't know of the second bug, which is actually completly independent from
the first one.

I checked with a 16 images album, recursive mode and All mode works fine, but
with a 2,000 images, the recursive mode only has 201 images.
I tried multiple times with different albums, always 201 images.

Same bug with latest digikam 0.9.5-svn

Some limits with ImageInfoJob ?


--
Configure bugmail: http://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[yann LE HIR-2]

http://bugs.kde.org/show_bug.cgi?id=176086





--- Comment #5 from yann LE HIR <yann lehir gmail com>  2008-11-25 23:54:52 ---
Ok, I figured it out. Each time a slotItemsInfo is thrown, ImageInfoAlbumsJob
changes of album, it should only change of album when signalCompleted is
thrown.


--
Configure bugmail: http://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[yann LE HIR-2]

http://bugs.kde.org/show_bug.cgi?id=176086


yann LE HIR yann lehir gmail com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #28821|0                           |1
        is obsolete|                            |




--- Comment #6 from yann LE HIR <yann lehir gmail com>  2008-11-25 23:56:06 ---
Created an attachment (id=28827)
 --> (http://bugs.kde.org/attachment.cgi?id=28827)
patch


--
Configure bugmail: http://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

http://bugs.kde.org/show_bug.cgi?id=176086





--- Comment #7 from Gilles Caulier <caulier gilles gmail com>  2008-12-03 16:49:03 ---
Andi,

yann has updated patch for you...

Gilles


--
Configure bugmail: http://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

http://bugs.kde.org/show_bug.cgi?id=176086





--- Comment #8 from Andi Clemens <andi clemens gmx net>  2008-12-03 17:03:36 ---
Sounds fine, works fine... I will commit it.

Andi


--
Configure bugmail: http://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

http://bugs.kde.org/show_bug.cgi?id=176086


Andi Clemens andi clemens gmx net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




--- Comment #9 from Andi Clemens <andi clemens gmx net>  2008-12-03 17:06:07 ---
SVN commit 892133 by aclemens:

patch by yann LE HIR: fix recursive slideshow

BUG:176086

 M  +1 -11     imageinfoalbumsjob.cpp  


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


--
Configure bugmail: http://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

https://bugs.kde.org/show_bug.cgi?id=176086


Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|0.10.0-svn                  |0.10.0




--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel