[Bug 185106] New: Advanced search is not saved/restored correctly

classic Classic list List threaded Threaded
11 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 185106] New: Advanced search is not saved/restored correctly

Bugzilla from andi.clemens@gmx.net
https://bugs.kde.org/show_bug.cgi?id=185106

           Summary: Advanced search is not saved/restored correctly
           Product: digikam
           Version: unspecified
          Platform: unspecified
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: NOR
         Component: Searches
        AssignedTo: [hidden email]
        ReportedBy: [hidden email]


Version:           0.10.0 (using 4.2.00 (KDE 4.2.0), Arch Linux)
Compiler:          gcc
OS:                Linux (i686) release 2.6.28-ARCH

When I save an advanced search and edit it again, some options are displayed
wrong. Also advanced search doesn't seem to work in general, I will explain
later...

--
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
Reply | Threaded
Open this post in threaded view
|

[Bug 185106] Advanced search is not saved/restored correctly

Bugzilla from andi.clemens@gmx.net
https://bugs.kde.org/show_bug.cgi?id=185106


Andi Clemens <[hidden email]> changed:

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




--
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
Reply | Threaded
Open this post in threaded view
|

[Bug 185106] Advanced search is not saved/restored correctly

Bugzilla from andi.clemens@gmx.net
In reply to this post by Bugzilla from andi.clemens@gmx.net
https://bugs.kde.org/show_bug.cgi?id=185106





--- Comment #1 from Andi Clemens <andi clemens gmx net>  2009-02-21 11:22:23 ---
Created an attachment (id=31515)
 --> (http://bugs.kde.org/attachment.cgi?id=31515)
wrong options

This screenshot shows the differences.
On the left is the screen when I first created the search.
On the right is the screen when I edit the search.
As you can see, the matching conditions as well as the logical conjunction
between the two groups changed.

But this is not the only issue. When I create the above search, it doesn't work
as expected. I wanted to create a search where images are returned that have
neither tag A nor tag B in it. But I always get images with only tag A assigned
to them.

Another example: If I create a search like this:
(all conditions met: search all albums: tag A) AND
(all conditions met: search all albums: tag B)

I still get only the images with tag A assigned to them.
Something is broken here...

Andi

--
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
Reply | Threaded
Open this post in threaded view
|

[Bug 185106] Advanced search is not saved/restored correctly

Bugzilla from andi.clemens@gmx.net
In reply to this post by Bugzilla from andi.clemens@gmx.net
https://bugs.kde.org/show_bug.cgi?id=185106





--- Comment #2 from Andi Clemens <andi clemens gmx net>  2009-02-21 11:27:40 ---
1. Another note on the advanced search dialog: I think it is way too big.
We should at least put every search group into a tab, it is so hard to read.

2. Why does the first group header has a different arrangement of the
conditions than the second? These headers are wasting a lot of space anyway.

What do you think? Although there is a huge blue header, I still find it hard
to distinguish between the two search groups.
It would really help to have tabs here, mainly because a single search group
itself can be 2 pages long, when every category has been expanded.

Andi

--
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
Reply | Threaded
Open this post in threaded view
|

[Bug 185106] Advanced search is not saved/restored correctly

Marcel Wiesweg
In reply to this post by Bugzilla from andi.clemens@gmx.net
https://bugs.kde.org/show_bug.cgi?id=185106





--- Comment #3 from Marcel Wiesweg <marcel wiesweg gmx de>  2009-02-21 17:42:47 ---
SVN commit 929571 by mwiesweg:

Simplify: complex and buggy code is not necessary here (other parts are complex
enough)

CCBUG: 185106

 M  +1 -14     searchgroup.cpp  


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

--
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
Reply | Threaded
Open this post in threaded view
|

[Bug 185106] Advanced search is not saved/restored correctly

Marcel Wiesweg
In reply to this post by Bugzilla from andi.clemens@gmx.net
https://bugs.kde.org/show_bug.cgi?id=185106


Marcel Wiesweg <[hidden email]> changed:

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




--- Comment #4 from Marcel Wiesweg <marcel wiesweg gmx de>  2009-02-21 17:42:52 ---
SVN commit 929572 by mwiesweg:

Add missing implementation for "OrNot" (which was not properly written or read
as a consequence)

BUG: 185106

 M  +5 -0      searchxml.cpp  


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

--
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
Reply | Threaded
Open this post in threaded view
|

[Bug 185106] Advanced search is not saved/restored correctly

Bugzilla from andi.clemens@gmx.net
In reply to this post by Bugzilla from andi.clemens@gmx.net
https://bugs.kde.org/show_bug.cgi?id=185106


Andi Clemens <[hidden email]> changed:

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




--- Comment #5 from Andi Clemens <andi clemens gmx net>  2009-02-21 18:05:09 ---
There is still a problem:
When I edit a saved search, the changes go into "Current Search", not the one I
edited.

Andi

--
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
Reply | Threaded
Open this post in threaded view
|

[Bug 185106] Advanced search is not saved/restored correctly

Marcel Wiesweg
In reply to this post by Bugzilla from andi.clemens@gmx.net
https://bugs.kde.org/show_bug.cgi?id=185106





--- Comment #6 from Marcel Wiesweg <marcel wiesweg gmx de>  2009-02-21 18:11:23 ---
SVN commit 929591 by mwiesweg:

Backport:
Add missing implementation for "OrNot" (which was not properly written or read
as a consequence)

CCBUG: 185106

 M  +5 -0      searchxml.cpp  


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

--
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
Reply | Threaded
Open this post in threaded view
|

[Bug 185106] Advanced search is not saved/restored correctly

Marcel Wiesweg
In reply to this post by Bugzilla from andi.clemens@gmx.net
https://bugs.kde.org/show_bug.cgi?id=185106


Marcel Wiesweg <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]




--- Comment #7 from Marcel Wiesweg <marcel wiesweg gmx de>  2009-02-21 18:31:28 ---
Works for me. I select any stored search except Current/Last Search, click on
Edit Stored Search's Edit button, edit, click ok. The stored search is edited,
Current/Last Search is untouched.

--
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
Reply | Threaded
Open this post in threaded view
|

[Bug 185106] Advanced search is not saved/restored correctly

Bugzilla from andi.clemens@gmx.net
In reply to this post by Bugzilla from andi.clemens@gmx.net
https://bugs.kde.org/show_bug.cgi?id=185106


Andi Clemens <[hidden email]> changed:

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




--- Comment #8 from Andi Clemens <andi clemens gmx net>  2009-02-21 18:44:47 ---
I recompiled everything again (make clean) and now it seems to work here, too.
I will watch this issue a little bit more...

Andi

--
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
Reply | Threaded
Open this post in threaded view
|

[Bug 185106] Advanced search is not saved/restored correctly

Gilles Caulier-4
In reply to this post by Bugzilla from andi.clemens@gmx.net
https://bugs.kde.org/show_bug.cgi?id=185106


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