Dotclear

Vous n'êtes pas identifié(e).

Annonce

#1 2017-03-22 13:39:32

adjaya
Membre
Lieu : Provence - Luberon
Inscription : 2006-09-05
Site Web

function getFeedID($rs) avec deux return

Bonjour
dans inc/core/class.dc.rs.extention
ligne 365

	public static function getFeedID($rs)
	{
		return 'urn:md5:'.md5($rs->core->blog->uid.$rs->post_id);

		$url = parse_url($rs->core->blog->url);
		$date_part = date('Y-m-d',strtotime($rs->post_creadt));

		return 'tag:'.$url['host'].','.$date_part.':'.$rs->post_id;
	}

et ligne 709

	public static function getFeedID($rs)
	{
		return 'urn:md5:'.md5($rs->core->blog->uid.$rs->comment_id);

		$url = parse_url($rs->core->blog->url);
		$date_part = date('Y-m-d',strtotime($rs->comment_dt));

		return 'tag:'.$url['host'].','.$date_part.':'.$rs->comment_id;
	}

Le code après le premier return de chaque fonction ne seras jamais évalué, non?


Photo, Art et Création Numérique : http://benoit-grelier.photo7.fr/

Hors ligne

#2 2017-03-22 15:18:04

Franck
Footer de merde
Lieu : Paris
Inscription : 2004-11-09
Site Web

Re : function getFeedID($rs) avec deux return

Ah oui tout à fait, c'est du code qui ne sert à rien.


Dotclear addicted since 2004

Hors ligne

#3 2017-03-22 17:04:57

adjaya
Membre
Lieu : Provence - Luberon
Inscription : 2006-09-05
Site Web

Re : function getFeedID($rs) avec deux return

Franck a écrit :

Ah oui tout à fait, c'est du code qui ne sert à rien.

pourrait servir éventuellement:

 public static function getFeedID($rs, $id = 'urn')
{
    if ($id == 'urn')	{
               return 'urn:md5:'.md5($rs->core->blog->uid.$rs->post_id);}
    }
     else
    {
  		$url = parse_url($rs->core->blog->url);
  		$date_part = date('Y-m-d',strtotime($rs->post_creadt));

  		return 'tag:'.$url['host'].','.$date_part.':'.$rs->post_id;
    }
}

Question de goût sûrement, je trouve le deuxième return plus parlant.


Photo, Art et Création Numérique : http://benoit-grelier.photo7.fr/

Hors ligne

#4 2017-03-23 06:52:49

Franck
Footer de merde
Lieu : Paris
Inscription : 2004-11-09
Site Web

Re : function getFeedID($rs) avec deux return

Certes mais est-ce qu'on a besoin d'un truc « parlant » pour ça ? J'suis pas certain :-)


Dotclear addicted since 2004

Hors ligne

#5 2017-03-23 10:31:53

adjaya
Membre
Lieu : Provence - Luberon
Inscription : 2006-09-05
Site Web

Re : function getFeedID($rs) avec deux return

Franck a écrit :

Certes mais est-ce qu'on a besoin d'un truc « parlant » pour ça ? J'suis pas certain :-)

code actuel:

<id>urn:md5:71500e40c5a5bc737388829eac2a5295</id>

alternative:

<id>tag:domaine.lo,2012-02-23:post268</id>

Perso, pour mes projets en cours j'ai retenu la deuxième option.

Je sais que ca vas pas casser la troisième patte à un canard, mais je trouve que le code humanisé est plus confortable à consulter.
Après, pour ce cas de figure je ne sais pas si les applications de lecture de flux en tienne compte!?


Photo, Art et Création Numérique : http://benoit-grelier.photo7.fr/

Hors ligne

#6 2017-03-23 13:55:34

Franck
Footer de merde
Lieu : Paris
Inscription : 2004-11-09
Site Web

Re : function getFeedID($rs) avec deux return

adjaya a écrit :

Après, pour ce cas de figure je ne sais pas si les applications de lecture de flux en tienne compte!?

That's the point!

En l'espèce aucune importance pour les clients des flux, c'est un ID à usage interne.


Dotclear addicted since 2004

Hors ligne

Vous n'êtes pas identifié(e).

Pied de page des forums

Sites map