Impossibility to indicate our own conditions in jControllerDaoCrud

[laurentj]

We cannot specify conditions to list records without overloading entirely index() method. jDaoConditions object should be passed as a parameter to _index(), or an other protected method should be provided.

[bastnic]

Voici un patch. Exemple d'utilisation :

	protected function _indexAddConditions($dao, $cond) {
		if(!jAcl::check("admin.client","ADMIN")){
			$user = jAuth::getUserSession();
			$cond->addCondition('userid','!=',$user->login);
		}
	}

[laurentj]

Je pense qu'il vaut mieux migrer le foreach $this->propertiesForRecordsOrder dans la nouvelle méthode (que l'on nommerait du coup plutôt _indexSetConditions). Ainsi, si on veut garder ces orders, on peut faire appels à la méthode d'origine, sinon on peut aussi changer toutes les conditions et les orders.

De plus, il ne faut pas oublier de changer le countAll en countBy.

Par contre je ne suis pas sûr qu'il faille passer l'objet $dao. Tu as un cas d'utilisation où cela serait utile ?

[bastnic]

Pour le dao je l'ai mis au cas où sans vraiment y réfléchir me disant que c'était en rapport avec les conditions mais c'est vrai que pour l'instant pas encore eu besoin.

J'ai normalement tout corrigé. Si on veut juste ajouter des conditions en gardant l'ordre par propertiesForRecordsOrder :

protected function _indexSetConditions($cond) {
    parent::_indexSetConditions($cond);
    $cond->addCondition('id','!=',4);	
}

Pour tout écraser :

protected function _indexSetConditions($cond) {
}

[laurentj]

Parfait !

[laurentj]

Fixed in the trunk and 1.0.x branch. svn 799.

Thanks for this patch.

[bastnic]

Merci pour la validation et le checkin.

ça n'a aucun sens de mettre en milestone 1.0.1 non ?