Changeset 1286

Timestamp:

Apr 28, 2006, 6:38:36 AM (18 years ago)

Author:

rvelices

Message:

merge -r1285 in from branch-1_6 to trunk
bug 359: transform rating html A links to FORM because some bots rate
pictures (even if rel="nofollow" is set)

Location:

trunk

Files:

• include/picture_rate.inc.php (modified) (3 diffs)
• language/en_UK.iso-8859-1/common.lang.php (modified) (2 diffs)
• language/fr_FR.iso-8859-1/common.lang.php (modified) (2 diffs)
• picture.php (modified) (1 diff)
• template/yoga/image.css (modified) (1 diff)
• template/yoga/picture.tpl (modified) (1 diff)
• template/yoga/theme/clear/theme.css (modified) (1 diff)
• template/yoga/theme/dark/theme.css (modified) (1 diff)

trunk/include/picture_rate.inc.php

@@ -55 +55 @@
   }
 
+  $user_rate = null;
   if ($conf['rate_anonymous'] or is_autorize_status(ACCESS_CLASSIC) )
   {
@@ -79 +80 @@
       {
         $row = mysql_fetch_array($result);
-        $sentence = $lang['already_rated'];
-        $sentence.= ' ('.$row['rate'].'). ';
-        $sentence.= $lang['update_rate'];
-      }
-      else
-      {
-        $sentence = $lang['never_rated'].'. '.$lang['Rate'];
+        $user_rate = $row['rate'];
       }
     }
-    else
-    {
-      $sentence = $lang['never_rated'].'. '.$lang['Rate'];
-    }
+
     $template->assign_block_vars(
       'rate',
       array(
-        'CONTENT' => $value,
-        'SENTENCE' => $sentence
+        'SENTENCE' =>isset($user_rate) ? l10n('update_rate') : l10n('new_rate'),
+        'F_ACTION' => add_url_params(
+                        $url_self,
+                        array('action'=>'rate')
+                      )
         )
       );
 
     $template->assign_block_vars('info_rate', array('CONTENT' => $value));
-
-    $template->assign_vars(
-      array(
-        'INFO_RATE' => $value
-        )
-      );
 
     foreach ($conf['rate_items'] as $num => $mark)
@@ -114 +103 @@
         array(
           'OPTION'    => $mark,
-          'URL'       => add_url_params(
-                          $url_self,
-                          array(
-                            'action'=>'rate',
-                            'rate'=>$mark
-                          )
-                        ),
           'SEPARATOR' => ($num > 0 ? '|' : ''),
           )
         );
+      if (isset($user_rate) and $user_rate==$mark)
+      {
+        $template->assign_block_vars('rate.rate_option.my_rate', array() );
+      }
+      else
+      {
+        $template->assign_block_vars('rate.rate_option.not_my_rate', array() );
+      }
     }
   }

trunk/language/en_UK.iso-8859-1/common.lang.php

@@ -131 +131 @@
 $lang['all'] = 'all';
 $lang['all_categories'] = 'all categories';
-$lang['already_rated'] = 'You\'ve already rated this item';
 $lang['ascending'] = 'ascending';
 $lang['author(s) : %s'] = 'author(s) : %s';
@@ -245 +244 @@
 $lang['nb_row_per_page'] = 'Number of rows per page';
 $lang['nbm_unknown_identifier'] = 'Unknown identifier';
-$lang['never_rated'] = 'You\'ve never rated this item';
 $lang['new_password'] = 'New password';
 $lang['new_password_hint'] = 'You only have to give a password if you wish to change it.';
 $lang['next_page'] = 'Next';
+$lang['new_rate'] = 'Rate this picture';
 $lang['no'] = 'No';
 $lang['no_category'] = 'Home';

trunk/language/fr_FR.iso-8859-1/common.lang.php

@@ -130 +130 @@
 $lang['all'] = 'tout';
 $lang['all_categories'] = 'toutes les catégories';
-$lang['already_rated'] = 'Vous avez déjà voté pour cette image';
 $lang['ascending'] = 'croissant';
 $lang['author(s) : %s'] = 'auteur(s) : %s';
@@ -244 +243 @@
 $lang['nb_row_per_page'] = 'Nombre de lignes par page';
 $lang['nbm_unknown_identifier'] = 'Identifiants inconnus';
-$lang['never_rated'] = 'Vous n\'avez jamais voté pour cette image';
 $lang['new_password'] = 'Nouveau mot de passe';
 $lang['new_password_hint'] = 'Vous n\'avez à confirmer votre mot de passe que si vous désirez en changer.';
+$lang['new_rate'] = 'Votez pur cette image';
 $lang['next_page'] = 'Suivant';
 $lang['no'] = 'Non';

trunk/picture.php

@@ -166 +166 @@
     {
       include_once(PHPWG_ROOT_PATH.'include/functions_rate.inc.php');
-      rate_picture($page['image_id'], $_GET['rate']);
+      rate_picture($page['image_id'],
+          isset($_POST['rate']) ? $_POST['rate'] : $_GET['rate'] );
       redirect($url_self);
     }

trunk/template/yoga/image.css

@@ -108 +108 @@
 }
 
+.rateButton, .rateButtonSelected {
+  background-color:transparent;
+  padding:0;
+  border:0;
+}
+
+.rateButton {
+  cursor: pointer;
+}
+
+.rateButtonSelected {
+  color:inherit;
+  font-weight:bold;
+  font-size:120%;
+}
+
 #comments {
     text-align: left;

trunk/template/yoga/picture.tpl

@@ -154 +154 @@
 
 <!-- BEGIN rate -->
-<p>
-{rate.SENTENCE} :
+<form action="{rate.F_ACTION}" method="post">
+<div>{rate.SENTENCE} :
 <!-- BEGIN rate_option -->
-{rate.rate_option.SEPARATOR} <a href="{rate.rate_option.URL}" rel="nofollow" {TAG_INPUT_ENABLED}>{rate.rate_option.OPTION}</a>
+{rate.rate_option.SEPARATOR}
+<!-- BEGIN my_rate -->
+<input type="button" name="rate" value="{rate.rate_option.OPTION}" class="rateButtonSelected" />
+<!-- END my_rate -->
+<!-- BEGIN not_my_rate -->
+<input type="submit" name="rate" value="{rate.rate_option.OPTION}" class="rateButton" />
+<!-- END not_my_rate -->
 <!-- END rate_option -->
-</p>
+</div>
+</form>
 <!-- END rate -->
 

trunk/template/yoga/theme/clear/theme.css

@@ -54 +54 @@
 
 /* links */
-A {
+A, .rateButton {
     color: #005e89;
     background: transparent;

trunk/template/yoga/theme/dark/theme.css

@@ -84 +84 @@
 
 /* links */
-A {
+A, .rateButton {
   color: #FFFFFF;
 }