Ticket #3549 #3716

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
5 participants
Contributor

backbone87 commented Dec 23, 2011

Implementation of ticket #3549

Toflar commented on 3c66bf5 Dec 23, 2011

The DB class has an abstraction in the drivers for FIND_IN_SET(), can't we use this somehow?

Owner

backbone87 replied Dec 23, 2011

Nope. The findInSet() from the Database class is designed to be used for the ORDER BY clause.

Owner

leofeyer commented May 10, 2012

@contao/workgroup-core Do we need this? What is the advantage? Does the patch work?

Contributor

psi-4ward commented May 10, 2012

For me its okay to implement these functionality, but i have not tested it with the 2.11.3 yet.

Owner

leofeyer commented May 10, 2012

What is the advantage at all? Is Andreas' current solution not efficient?

Contributor

backbone87 commented May 10, 2012

Look at the original ticket #3549
This solution supports filtering on comma separated lists for 1:n and n:n relations, which can be handled natively by MySQLs FIND_IN_SET

Contributor

psi-4ward commented May 10, 2012

@backbone87 könnte es nicht auch mit diesem Trick gelöst werden: http://de.contaowiki.org/DCA:_Sortierung_anhand_einer_Kindtabelle

Sprich, du legst ein "virtuelles" Feld an was als key ein Subquery ist?

Contributor

backbone87 commented May 10, 2012

@psi-4ward in dem ticket gehts, aber um Filterung und nicht Sortierung. Das hatten wir doch alles schon mal besprochen dachte ich?

Contributor

psi-4ward commented May 10, 2012

Es geht auch um die Idee dahinter, liese sich das nicht auch auf die Filterung übertragen?

Contributor

backbone87 commented May 10, 2012

Nein, da man nirgends für die Filterung individuelles SQL absetzen kann. Davon abgesehen, finde ich die Sache mit SQL im Sortierarray ziemlich hacky, nichts was ich als "Teil der API" betrachten würde, da es einfach zu unberechenbar ist.

Member

Toflar commented Aug 9, 2012

@contao/workgroup-core any updates?

Owner

leofeyer commented Sep 25, 2012

We have decided not to implement the feature, because there are ways to accomplish it using callbacks and we don't want to have more undocumented features which only a few users need. BTW, what did you try to do here:

$options = explode(',', implode(',', $options));

leofeyer closed this Sep 25, 2012

Contributor

backbone87 commented Sep 25, 2012

array('blub,blab', 'blib,blob')

Und bitte nenne mir die callbacks mit denen ich das umsetzen kann und seit welcher Version diese Verfügbar sind.

Weiterhin sehe ich das nicht als "undocumented feature", sondern im jetzigen Zustand nicht dokumentierte, nicht unterstützte Ausnahme im Filtersystem. Nirgendswo steht, das der Filter nicht Konsistent auf "multi-valued" Feldern funktioniert.

Member

aschempp commented Sep 26, 2012

Nach unseren Überlegungen müsste genau dasselbe auch mit einem onload_callback und list > sorting > root Array zu lösen sein.

Contributor

backbone87 commented Sep 26, 2012

Nach erstem schauen, ist dies mit 2.11.5 DC_Table NICHT möglich.

  1. Wie bekomme ich mein Filter-Dropdown in den View rein?
  2. list.sorting.root bezieht sich nur auf die numerischen PK-IDs, nicht auf ein beliebiges filterbares Feld
Member

aschempp commented Sep 26, 2012

THIS IS GETTING FUN...
While talking to @backbone87, we noticed that this feature has already been implemented!! You need to set ['eval']['csv'] = true;, and the core does even more than this pull request suggests (auto-save as CSV instead of serialized).

I suggest the feature should be documented ;-)

Contributor

backbone87 commented Sep 26, 2012

I have looked at the code and ['eval']['csv'] = ',' // your delimiter to use should be the way to use it in current versions

Owner

leofeyer commented Sep 27, 2012

Kann das Ticket also geschlossen bleiben?

Member

aschempp commented Sep 27, 2012

Absolut!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment