Discussion :

[bugs_budy]

Bonjour,

Je souhaiterais tronquer les pseudos de mes utilisateurs.

En recherchant sur le net j'ai trouvé une fonction pour tronquer un texte :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
function Tronquer_Texte($texte, $longeur_max)
{
    if (strlen($texte) > $longeur_max)
    {
        $texte = substr($texte, 0, $longeur_max);
        $dernier_espace = strrpos($texte, "");
        $texte = substr($texte, 0, $dernier_espace)."...";
    }
 
    return $texte;
}
 
echo Tronquer_Texte("Voici un exemple de texte tronqué.", 20);

La ligne de code qui retourne le pseudo de l'utilisateur est la suivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$result .= "<td valign='top'>".$this->_s4j->GetUser( $row )."</td>";

Comment intégrer la fonction ou la variable qui tronque le pseudo dans la variable qui retourne le pseudo ?

Je vous remercie

[sharrascript]

bonjour,

Voilà comment utiliser ta fonction:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$result .= "<td valign='top'>".Tronquer_Texte($this->_s4j->GetUser( $row ), 20)."</td>";

Un exemple pour 20 caractère max

++

[bugs_budy]

Bonjour,

Voici ce que me retourne le serveur :

Fatal error: Call to undefined function: tronquer_texte() in c:\program files\easyphp1-8\www\generationelectro\modules\mod_s4jnewusers.php on line 81

Et voici mon code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
function Tronquer_Texte($texte, $longeur_max)
{
    if (strlen($texte) > $longeur_max)
    {
        $texte = substr($texte, 0, $longeur_max);
        $dernier_espace = strrpos($texte, "");
        $texte = substr($texte, 0, $dernier_espace)."...";
    }
 
    return $texte;
}
 
		function Show() {
			$columns_count	= $this->_params->get('columns_count', 5); 
			$rows			= $this->GetDataRows();
 
			$tr 			= 1;
			$ind			= 0;
			$rows_count		= count($rows); //NOTE: Improve later
 
			$result 		= "<table border='0'>"; // TODO: Change template
			foreach ($rows as $row) {
				if( $tr == 1) 
					$result .= "<tr>";
				$result .= "<td valign='top'>".Tronquer_Texte($this->_s4j->GetUser( $row ), 20)."</td>";
...

Merci

[sharrascript]

bonsoir,

tronquer_texte() != Tronquer_Texte()

Attention à la casse

[julp]

Le nom des fonctions n'est pas sensible à la casse contrairement aux variables. Le problème est donc ailleurs : ici il ne s'agit probablement plus d'une fonction mais d'une méthode donc son appel doit être rectifié ou alors il faut la déplacer pour qu'elle ne soit plus située dans la déclaration de la classe.

Edit : pour PHP 5, vous avez la possibilité de l'utiliser comme méthode de classe (statique)

[sharrascript]

Salut Julp,

Les fonctions ne sont plus sensibles à la casse depuis PHP5??
Parce qu'en PHP4 elle l'était encore, non??

Je me trompe peut être, mais je croyais vraiment qu'elles avaient les mêmes restrictions que les variables...

J'vais me replonger dans mes bouquins, moi^^

[sharrascript]

Re en effet,

Apparemment les fonctions sont insensibles à la casse, mais on trouve encore sur certains tutos du net des règles qui expliques que les fonctions sont sensibles à la casse. Mais bon, comme le dit un site référence:

Note

Les noms de fonctions sont insensibles à la casse, et il est généralement admis que les fonctions doivent être appelée avec le nom utilisé dans leur déclaration, y compris la casse.

Donc désolé, et en effet le problème vient surement de l'endroit où tu déclare ta fonction bugs_budy.

++

[bugs_budy]

Merci pour vos réponses. Etant vraiment débutant en php, je ne suis pas en mesure de savoir ou placer la fonction...

Voici le code de ma page :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
<?php
 
/** ensure this file is being included by a parent file */
defined( '_VALID_MOS' ) or die( 'Direct Access to this location is not allowed.' );
 
 
global $mosConfig_absolute_path, $mosConfig_lang;
$subpath = "/modules/mod_s4jnewusers";
require_once( $mosConfig_absolute_path .$subpath."/helper.php" ); 
 
//Language definitions
if (file_exists($mosConfig_absolute_path.$subpath."/language/".$mosConfig_lang.".php")){ 
	include($mosConfig_absolute_path.$subpath."/language/".$mosConfig_lang.".php");
} else { 
	include($mosConfig_absolute_path.$subpath."/language/english.php");
}
 
//if modul was copied, check the class
if( !class_exists("s4jNewUsers") ) {
 
	class s4jNewUsers {
		var $_s4j	= null;
		var $_db	= null;
		var $_params= null;
 
 
		/**
		 * Construnctor
		 */
		function s4jNewUsers( &$params ) {
			global $database;
			$this->_db 		= $database;
			$this->_s4j		= new s4jLibNewUsers( $params, 2,  "mod_s4jnewusers" );
			$this->_params	= & $params;
		}
 
 
		function GetDataRows() {
			$max_user_shown	= $this->_params->get('max_user_shown', 3); 
 
			$sql = "SELECT DISTINCT
						u.id, u.username, u.name,
						s.userid as s4jonline";
			$sql .= $this->_s4j->GetFields();
			$sql .= " FROM
						( #__users u INNER JOIN #__comprofiler c ON u.id = c.user_id AND u.block =0 AND c.confirmed =1 AND c.approved =1 ) 
						LEFT JOIN #__session s ON s.userid = u.id AND s.guest = 0";
			$sql .= $this->_s4j->GetFilter("c", "u", "WHERE");
			$sql .=	" ORDER BY registerdate DESC
					  LIMIT 0, " . $max_user_shown;
 
			$this->_db->setQuery( $sql );
			$rows = $this->_db->loadObjectList();
			return $rows;
		}
 
 
 
		function Show() {
			$columns_count	= $this->_params->get('columns_count', 5); 
			$rows			= $this->GetDataRows();
 
			$tr 			= 1;
			$ind			= 0;
			$rows_count		= count($rows); //NOTE: Improve later
 
			$result 		= "<table border='0'>"; // TODO: Change template
			foreach ($rows as $row) {
				if( $tr == 1) 
					$result .= "<tr>";
				$result .= "<td valign='top'>".$this->_s4j->GetUser( $row )."</td>";
 
				if($tr == $columns_count) {
					$result .= "</tr>";
					$tr=1;
				} else if( $ind == $rows_count ) {
					for($j = $tr; $j < $columns_count; $j++) {
						$result .= "<td style='display:none;'>&nbsp;</td>";
					}
					$result .= "</tr>";
 
				} else {
					$tr++;
				}
				$ind++;
 
			}
 
			$result 		.= "</table>";
			return $result;
		}
	}
 
	function s4jNewUsersCaching( & $params ){
		$s4NewUsers = new s4jNewUsers( $params ); 
		echo $s4NewUsers->Show();
	}
}
 
 
 
$enablecache = $params->get('enablecache', 0);
if( $enablecache == 1 ) {
	$cache =& mosCache::getCache( 'mod_s4jnewusers' );
	$cache->call( 's4jNewUsersCaching',  $params );
} else {
	s4jNewUsersCaching( $params );
} 
 
?>

Et la fonction que je souhaite ajouter :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
function Tronquer_Texte($texte, $longeur_max)
{
    if (strlen($texte) > $longeur_max)
    {
        $texte = substr($texte, 0, $longeur_max);
        $dernier_espace = strrpos($texte, "");
        $texte = substr($texte, 0, $dernier_espace)."...";
    }
 
    return $texte;
}
 
echo Tronquer_Texte("Voici un exemple de texte tronqué.", 20);

Merci

[bugs_budy]

Le développeur de cette application m'a répondu. Il faut faire une modification dans un fichier externe de cette fonction : (helper.php)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
 
function GetUserName( &$row, $format ) {
        switch ($format) {
            case 0 :
                return $row->username;
            break;
            case 1 :
                return $row->username." (".$row->name.")";
            break;
            case 2 :
                return $row->name;
            break;
            case 3 :
                return $row->name." (".$row->username.")";
            break;
        }
        return "";
    }

J'ai trouvé celle ci pour tronquer les pseudos :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 function Truncate_Text($text, $max_lenght)
{
if (strlen($text) > $max_lenght)
{
$text = substr($text, 0, $max_lenght);
$last_space = strrpos($text, "");
$text = substr($text, 0, $last_space)."...";
}
 
return $text;
}
 
echo Truncate_Text("Your example text", 20);

Et la ligne pour appeler les 2 fonctions (pseudo + tronquage) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$result .= "<td valign='top'>".Tronquer_Texte($this->_s4j->GetUser( $row ), 20)."</td>";

J'ai essayé plein de possibilité mais je suis complètement largué. A chaque fois le serveur me dit qu'il ne connait pas la fonction Truncate_Text

Merci

[opware2000]

Simple question : tu as inclus le fichier contenant ta nouvelle fonction ?

[Rakken]

Pour revenir sur les majuscules dans le nom de fonction, Tronquer_Texte, ca n'est pas un nom terrible. L'usage voudrai plutot soit "tronquerTexte" ou "tronquer_texte".
Les noms qui commencent en majuscules sont plutot réservé pour des noms de classes (et les noms intégralement en majuscule sont pour les constantes)

Sinon, a l'instar de opware2000, je poserai la même question, le fichier ou se trouve ta fonction est-il bien inclus ?

[bugs_budy]

Oui il est bien inclus dans le fichier mod_s4jnewusers.php avec l'appel de la fonction :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

require_once( $mosConfig_absolute_path .$subpath."/helper.php" );

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$result .= "<td valign='top'>".truncate_text($this->_s4j->GetUser( $row ), 10)."</td>";

J'ai modifié la variable comme le conseillait Rakken.

L'auteur du script me dit de modifier la fonction GetUserName alors que pour afficher le pseudo on sert de la fonction GetUser. Une erreur de sa part ?

Bref, voici les trois fonctions de helper.php :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
	function truncate_text($text, $max_lenght)
{
if (strlen($text) > $max_lenght)
{
$text = substr($text, 0, $max_lenght);
$last_space = strrpos($text, "");
$text = substr($text, 0, $last_space)."...";
}
 
return $text;
}

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
	function GetUserName( &$row, $format ) {
		switch ($format) {
			case 0 :
				return $row->username;
			break;
			case 1 :
				return $row->username." (".$row->name.")";
			break;
			case 2 :
				return $row->name;
			break;
			case 3 :
				return $row->name." (".$row->username.")";
			break;
		}
		return "";
	}

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
 
	function GetUser( &$row ) {
		$result = $this->_out_format;
 
		$profillink = sefRelToAbs("index.php?option=com_comprofiler&task=userProfile&user=".$row->id);
		$displayname = $this->GetUserName( $row, $this->_format_name );
 
		//show tooltip if enable
		if( $this->_show_tooltip == 1 ) {
			$tooltip = $this->GetTooltip( $row );
			$profile = $this->mosToolTip($tooltip, $displayname, "", "", $displayname, $profillink, 1 ); // TODO: noch nciht fertig
		} else {
			$profile =  $this->link( $profillink, $displayname );
		}
		$result = str_replace("U%", $profile, $result);		
 
		if( $this->_cb_age > 0 ) {
			$result = str_replace("B%", $this->GetAge( $row, $this->_cb_age ), $result);
		} else {
			$result = str_replace("B%", "", $result);
		}
 
		//link to profile from avatar
		$avatarlink = "";
		if( $this->_cb_avatar > 0 ) {
			$avatarlink = $this->link( $profillink, $this->GetAvatar( $row, $this->_cb_avatar, false ) ); 
		}
		$result = str_replace("A%", $avatarlink, $result);
 
		if( $this->_cb_gender > 0 ) {
			$result = str_replace("G%", $this->GetGender( $row, $this->_cb_gender ), $result);
		} else {
			$result = str_replace("G%", "", $result);
		}
 
		if( $this->_show_online > 0 ) {
			$result = str_replace("O%", $this->GetOnlineStatus( $row ), $result );
		} else {
			$result = str_replace("O%", "", $result);
		}
 
		if( $this->_pms_link > 0 ) {
			$result = str_replace( "P%", $this->GetPmsLink($row), $result );
		} else {
			$result = str_replace( "P%", "", $result );
		}
 
		//hits
		$result = str_replace( "H%", ($this->_show_hits > 0)?( $row->hits ):(""), $result );
		//registration date
		$result = str_replace( "R%", ($this->_show_register > 0)?( $this->FormatDate($row->registerDate) ):(""), $result );
		//lastupdate date
		$result = str_replace( "L%", ($this->_show_lastupdatedate > 0)?( $this->FormatDate($row->lastupdatedate) ):(""), $result );
		//custom fields 1-3
		$result = str_replace( "C1%", ($this->_cb_show_custom1 > 0)?( $this->GetCustomField($row->s4jcustom1, $this->_cb_show_custom1, $this->_cb_custom1_image) ):(""), $result );
		$result = str_replace( "C2%", ($this->_cb_show_custom2 > 0)?( $this->GetCustomField($row->s4jcustom2, $this->_cb_show_custom2, $this->_cb_custom2_image) ):(""), $result );
		$result = str_replace( "C3%", ($this->_cb_show_custom3 > 0)?( $this->GetCustomField($row->s4jcustom3, $this->_cb_show_custom3, $this->_cb_custom3_image) ):(""), $result );
 
		return $result;
	}

Dois je faire un mix entre la fonction pour tronquer les pseudos et celle pour les afficher ?

[Rakken]

Bon, pour un truc comme "truncate_text", ce que je fais moi, généralement, c'est creer un fichier utils.php qui ressemble a ca (php5) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
<?php
class utils {
static public function truncate_text($text, $max_lenght) {
  if (strlen($text) > $max_lenght)
  {
      $text = substr($text, 0, $max_lenght);
      $last_space = strrpos($text, "");
      $text = substr($text, 0, $last_space)."...";
  }
  return $text;
}
}
?>

Apres, tu n'as plus qu'a inclure ton utils.php
Et tu l'appele comme ca :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$str = utils::truncate("blablabla", 3);

La notation :: signifie que tu appele une fonction de la classe utils sans l'instancier. Le mot clef static indique qu'il est possible d'appeler cette fonction de la sorte.
Essaie comme ca pour voir. Un nouveau fichier, que tu inclues, et au moment ou tu en a besoin, utils::truncate.