class.genericdb.php: Funktion delete der Item-Klasse

Gesperrt
HerrB
Beiträge: 6935
Registriert: Do 22. Mai 2003, 12:44
Wohnort: Berlin
Kontaktdaten:

class.genericdb.php: Funktion delete der Item-Klasse

Beitrag von HerrB »

Diese Funktion sollte IMHO entfallen (class.genericdb.php, Item-Klasse):

Code: Alles auswählen

	/**
	 * Deletes the current item
	 * 
	 * @return void
	 */
	function delete()
	{
		$this->_collectionInstance->delete($item->get($this->primaryKey));
	}
a) _collectionInstance ist nicht definiert
b) $item ist nicht definiert (das müsste $this sein)
c) Man dürfte es auch nicht korrigiert bekommen, da es IMHO keine Möglichkeit gibt, aus dem Item-Objekt heraus an die Collection-Instanz/das Collection-Objekt heranzukommen (trivialerweise, weil man sie nicht kennt...), in der die eigentliche delete-Methode definiert ist.

Gruß
HerrB
Bitte keine unaufgeforderten PMs oder E-Mails -> use da Forum!

Newsletter: V4.4.x | V4.6.0-15 (Module, Backend) | V4.6.22+
Standardartikelliste: V4.4.x | V4.6.x
http://www.contenido.org/forum/search.php | http://faq.contenido.org | http://www.communido.net
emergence
Beiträge: 10653
Registriert: Mo 28. Jul 2003, 12:49
Wohnort: Austria
Kontaktdaten:

Beitrag von emergence »

ich hab jetzt mal geschaut, ja sollte entfernt werden...
würde man die methode verwenden gäbe es sowieso nur ne fehlermeldung...
da die klasse selbst auch keine create methode hat, noch ein grund mehr das rauszunehmen...
*** make your own tools (wishlist :: thx)
HerrB
Beiträge: 6935
Registriert: Do 22. Mai 2003, 12:44
Wohnort: Berlin
Kontaktdaten:

Beitrag von HerrB »

Auskommentiert.

Gruß
HerrB
Bitte keine unaufgeforderten PMs oder E-Mails -> use da Forum!

Newsletter: V4.4.x | V4.6.0-15 (Module, Backend) | V4.6.22+
Standardartikelliste: V4.4.x | V4.6.x
http://www.contenido.org/forum/search.php | http://faq.contenido.org | http://www.communido.net
Gesperrt