Bug in tt_address

Schon wieder. Es ist nur eine kleine Klammer, aber mit großer Wirkung. Zumindest wurde bei mir nicht korrekt sortiert, wenn ich eine Eigenschaft ausgewählt habe (was anderes als name). Der Grund dafür ist die viel zu sehr verschachtelte Abfrage, um die Sortier-Reihenfolge festzulegen.
Was nicht funktioniert (class.tx_address_pi1.php Zeilen 140ff):

$this->conf['sortByColumn'] = $this->ffData['sortBy'] ?
	$this->ffData['sortBy'] :
		$this->conf['sortByColumn'] ?
			$this->conf['sortByColumn'] :
			'name';

Was aber funktioniert:

$this->conf['sortByColumn'] = $this->ffData['sortBy'] ?
	$this->ffData['sortBy'] :
		($this->conf['sortByColumn'] ?
			$this->conf['sortByColumn'] :
			'name');

Die fehlende Klammer sorgt dafür, dass die Sortierung immer auf „name“ gesetzt wird, egal was ausgewählt wurde.

3 Kommentare

  1. Torsten

    Vielen Dank für den Tip!
    Das hat es bei mir gebracht. Hab mich schon gewundert, warum die Sortierung nur immer auf Name ging.

    Gruß

  2. Andi

    Auch von mir ein Danke für den Tipp!

    Noch ein Hinweis dazu: Das Sortieren nach Gruppen funktioniert aber auch damit nicht. Habe zwar ‚addressgroup‘ in die checkSorting Methode aufgenommen aber das behebt das Problem auch nicht.

  3. Danke. Genau das war auch bei mir das Problem. Der Beitrag hat mir viel Zeit erspart, nach der Ursache zu suchen.