[TYPO3-core] RFC: Advanced BE search form

[Sebastian Kurfuerst]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi Michael,

first of all - nice :-)
The levels are nice.

Some comments I have:

    * the patch doesn't apply cleanly in current CVS due to Kasper's
      changes in DBint. I used beta2 to test it.
    * it would be nice if "Advanced query" was the default when "full
      search" is selected. I know this breaks BW "compatibility" up to
      a certain extent, but it is a new feature which should be
      "promoted".
    * for the checkbox where the current field can be negated, it
      would be nice to add some caption there (does the "title" tag
      work there?) to describe that it negates everything. It was a
      bit unclear to me at first.
    * If there is only one query left and you delete that one, there
      is no possibility to add another one again. Besides that, an SQL
      error appears then (logically).
    * There is a _serious_ problem when building queries - I don't get
      it to work. This means- I add one field, everything works
      nicely, I add a second line and change the type there. Then the
      page reloads and only the second line is still visible.

Because of the last issue, I couldn't test more advanced functions :-(


So, currently, I am afraid there is still some work to do. I hope you
have time for this or find someone who does it.
Btw, it definitely has to be inserted and published in the announcement!

Greets, Sebastian
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.1 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFD25z1ygDVWNTabbkRAvCzAJ9wd1BGqQDm0/8HSkyOpW4nRNgjbQCfc2/C
0vnUDIBE+FcIa57o3j6nhgE=
=3yYa
-----END PGP SIGNATURE-----