[Bug 163227] New: Searching for tag names in simple and advanced search doesn't work

------- 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=163227         
           Summary: Searching for tag names in simple and advanced search
                    doesn't work
           Product: digikam
           Version: unspecified
          Platform: unspecified
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: NOR
         Component: Searches
        AssignedTo: digikam-devel kde org
        ReportedBy: andi.clemens gmx net


Version:           0.9.4-rc (using 3.5.9, Arch Linux)
Compiler:          Target: i686-pc-linux-gnu
OS:                Linux (i686) release 2.6.25-ARCH

When using the simple or advanced search to find tag names, digiKam will show no results. For example if you have the family name "Clemens" and search for it, the SQL query will return no results. It seems to miss the "%" character in the LIKE query string.
_______________________________________________
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=163227         




------- Additional Comments From andi.clemens gmx net  2008-06-04 20:55 -------
Created an attachment (id=25124)
 --> (http://bugs.kde.org/attachment.cgi?id=25124&action=view)
fixes search problem

This patch fixes the following issues:

* simple and advanced search doesn't work for tag names
* lineedit widget in advanced search will expand, it was too short if window is
resized
* not needed search conditions are removed (searching for Tag doesn't need the
condition "contains" for example)

Gilles, Marcel, Arnd,

what do you think? Can I commit this?

Andi
_______________________________________________
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=163227         




------- Additional Comments From andi.clemens gmx net  2008-06-05 10:38 -------
I've found another problem: When editing an advanced search that has not any search rule with a lineedit field in it, the operator combobox becomes too small. If you edit a saved advanced search with a lineedit rule (like Tag name for example), the combobox is sized well.
I will also try to fix this issue with this patchset.
_______________________________________________
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=163227         




------- Additional Comments From andi.clemens gmx net  2008-06-05 10:40 -------
Additional info to #2:

If you generate a new advanced search, the operator comboxbox is always sized well, only saved advanced searches that are edited will behave strange sometimes.
_______________________________________________
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=163227         
andi.clemens gmx net changed:

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



------- Additional Comments From andi.clemens gmx net  2008-06-05 11:56 -------
Created an attachment (id=25133)
 --> (http://bugs.kde.org/attachment.cgi?id=25133&action=view)
patch 2

the patch fixes the following issue:

some items in the operator combobox were not readable, because the box was
resized on key change. Now the box is prepopulated with all operators first and
the size is saved on every key change.
_______________________________________________
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=163227         




------- Additional Comments From caulier.gilles gmail com  2008-06-05 12:01 -------
Andi,

This is relevant of this old entry (closed with KDE4 port) :

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

Gilles
_______________________________________________
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=163227         




------- Additional Comments From andi.clemens gmx net  2008-06-05 12:24 -------
Sure it might be fixed in KDE4 port, but not in the current stable version of digiKam. But the initial problem of this bugreport is not the widget behavior, but the search for tag names which didn't work anymore. The patch should solve this problem...
_______________________________________________
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=163227         




------- Additional Comments From caulier.gilles gmail com  2008-06-05 12:36 -------
Andi,

Why are you removed "LIKE" and "NLIKE" rules for tags searches ?

Gilles
_______________________________________________
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=163227         




------- Additional Comments From andi.clemens gmx net  2008-06-05 12:51 -------
It looks like they are never used.
If you search for a tag, you have the exact tagname in the combobox and there will never be a query for LIKE or NLIKE.
Like in album search. The only two operators allowed in there are EQ and NEQ.
So I removed the LIKE and NLIKE for tags from the rulesOpTable as well as from the kioslave (digikamsearch.cpp:489).
Searching for LIKE and NLIKE in "tag names" is still possible... but for "pure" tag searches I think it should be removed like it is removed in album search.
_______________________________________________
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=163227         
caulier.gilles gmail com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |marcel.wiesweg gmx de



------- Additional Comments From caulier.gilles gmail com  2008-06-05 12:59 -------
Marcel, what do you think about #8

Gilles
_______________________________________________
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=163227         




------- Additional Comments From andi.clemens gmx net  2008-06-05 13:05 -------
Ok, removing LIKE is wrong, if you want to find all images with a tag under "persons" (persons/andi, persons/gilles) you will need to have LIKE and NLIKE operators. But then it should also be added to album search, because otherwise it is inconsistent.
_______________________________________________
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=163227         
andi.clemens gmx net changed:

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



------- Additional Comments From andi.clemens gmx net  2008-06-05 16:26 -------
Created an attachment (id=25139)
 --> (http://bugs.kde.org/attachment.cgi?id=25139&action=view)
patch 3

fixes the following issues:

* simple and advanced search doesn't work for tag names
* new operators for album search added (LIKE, NLIKE)
* lineedit widget in advanced search will expand, it was too short if window is

resized
* some items in the operator combobox were not readable, because the box was
resized on key change. Now the box is prepopulated with all operators first and

the size is saved on every key change.
_______________________________________________
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=163227         




------- Additional Comments From caulier.gilles gmail com  2008-06-05 17:25 -------
Marcel,

> * new operators for album search added (LIKE, NLIKE)

==> is implemented in KDE4 ?

Gilles
_______________________________________________
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=163227         




------- Additional Comments From andi.clemens gmx net  2008-06-06 18:12 -------
Gilles,

I think it is implemented, I just tested the KDE4 version and LIKE for albums is definitely working there. If you select a folder ("2001"), it will search recursively.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Marcel Wiesweg]

------- 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=163227         




------- Additional Comments From marcel.wiesweg gmx de  2008-06-06 23:47 -------
In KDE4 I have called it InTree/NotInTree, and it's currently even the default behavior for the combo box tree views for albums and tags.
_______________________________________________
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=163227         




------- Additional Comments From andi.clemens gmx net  2008-06-06 23:57 -------
Is it not possible anymore in the KDE4 version to use the "NOT" operators? Like "does not contain" or "does not equal"?
I try to filter images that have NOT the tag "Persons" and that are NOT in folder "2007 and its subfolders". But there is no option in the new search dialog of digiKam 0.10... maybe I'm blind.

Marcel, what do you think of the new operators added in patch 3? I think they should be added to be consistent with the tag search.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Marcel Wiesweg]

------- 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=163227         




------- Additional Comments From marcel.wiesweg gmx de  2008-06-07 12:15 -------
No you're not blind. It's currently not in the UI. The backend can do that without any  problem. It is the problem to balance search capabilities with UI complexity. The least complex solution would be to add "Match none of the following conditions" in the label at the top. You could then add a search group for all "NOT" searches.

As to your patch, feel free to add the operators.
_______________________________________________
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=163227         
andi.clemens gmx net changed:

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



------- Additional Comments From andi.clemens gmx net  2008-06-07 12:44 -------
SVN commit 817988 by aclemens:

fixes the following issues:

* simple and advanced search doesn't work for "tag names"
* new operators for "album" search added (LIKE, NLIKE) to be consistant with "tag" search
* lineedit widget in advanced search needs to expand, it was too short if window is resized
* some items in the operator combobox were not readable, because the box was resized on key change. Now the box is prepopulated with all operators first and its size is restored on every key change.

BUG: 163227

 M  +2 -1      NEWS  
 M  +18 -9     digikam/searchwidgets.cpp  
 M  +12 -5     kioslave/digikamsearch.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=817988
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel