Merge lp:~malizor/ubuntu-fr-forum/correction-recherche into lp:ubuntu-fr-forum

Proposed by Nicolas Delvaux
Status: Merged
Merged at revision: 129
Proposed branch: lp:~malizor/ubuntu-fr-forum/correction-recherche
Merge into: lp:ubuntu-fr-forum
Diff against target: 34 lines (+6/-11)
1 file modified
search.php (+6/-11)
To merge this branch: bzr merge lp:~malizor/ubuntu-fr-forum/correction-recherche
Reviewer Review Type Date Requested Status
YoBoY (community) Approve
Review via email: mp+58701@code.launchpad.net

Description of the change

Correction du fil d'Ariane dans "Mes messages" et "Mes discussions" (depuis la page le profil)
(le nom d'utilisateur n'était pas affiché, il y avait juste "De ")

To post a comment you must log in.
130. By Nicolas Delvaux

Solution plus propre. Merci YoBoY !

131. By Nicolas Delvaux

Suppression d'une ligne inutile.

Revision history for this message
YoBoY (yoboy-leguesh) wrote :

Tout est corrigé avec cette version. Je suis vraiment une quiche des fois pour avoir pas vu que j'utilisais des variables non conservées :p

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'search.php'
2--- search.php 2011-04-21 10:02:31 +0000
3+++ search.php 2011-04-21 16:18:35 +0000
4@@ -573,24 +573,19 @@
5 $print_param = '';
6 switch ($search_type[1]) {
7 case "show_subscriptions":
8- // Fetch username of subscriber
9- $subscriber_id = $search_type[2];
10+ case "show_user_topics":
11+ case "show_user_posts":
12+ // Fetch username
13 $query = "SELECT username FROM {$db->prefix}users WHERE id={$search_type[2]}";
14 $result = $db->query($query) or error('Unable to fetch username of subscriber', __FILE__, __LINE__, $db->error());
15 if ($db->num_rows($result))
16- $subscriber_name = $db->result($result);
17+ $user_name = $db->result($result);
18 else
19 message($lang_common['Bad request']);
20- $print_param = pun_htmlspecialchars($subscriber_name);
21- break;
22- case "show_user_topics":
23- $print_param = pun_htmlspecialchars($search_set[0]['poster']);
24- break;
25- case "show_user_posts":
26- $print_param = pun_htmlspecialchars($search_set[0]['pposter']);
27+ $print_param = pun_htmlspecialchars($user_name);
28 break;
29 case "show_new":
30- $forum_actions = true;
31+ $forum_actions = true;
32 default :
33 $print_param = "";
34 }

Subscribers

People subscribed via source and target branches