Re: com web/master: Final attempt to fix pagination?: manage/user-notes.php

From: Date: Mon, 20 Apr 2020 05:32:04 +0000
Subject: Re: com web/master: Final attempt to fix pagination?: manage/user-notes.php
References: 1  Groups: php.webmaster 
Request: Send a blank email to php-webmaster+get-28600@lists.php.net to get a copy of this message
After third attempt I finally checked who added such a misleading comment and code that has followed it. Since I'm writing it you probably know already... Curse you, past-Maciej! :D W dniu 19.04.2020 o 12:59, Maciej Sobaczewski pisze:
Commit:    e471466e32ba7882ab08038de14a300deb8d1a8b
Author:    Sobak <msobaczewski@gmail.com>         Sun, 19 Apr 2020 12:59:43 +0200
Parents: b8cf248dc8fc004b41367dc9979af6fc778daf98 Branches: master
Link:       http://git.php.net/?p=web/master.git;a=commitdiff;h=e471466e32ba7882ab08038de14a300deb8d1a8b
Log: Final attempt to fix pagination? Changed paths: M manage/user-notes.php Diff: diff --git a/manage/user-notes.php b/manage/user-notes.php index da2145c..00c397f 100644 --- a/manage/user-notes.php +++ b/manage/user-notes.php @@ -375,19 +375,10 @@ if (!$action) {
               "number of votes placed in a small timeframe to help detect spam and other potential abuse.</p>\n".
               "<p>Also note that a <em>0.0.0.0</em> IP address indicates a client IP could not be resolved at the time of voting.</p>";
        }
-      // Temp debug
-      // @todo remove
-      echo '<!-- ';
-      print_r($_REQUEST, false);
-      print_r($search_votes, false);
-      echo ' -->';
-      if((isset($_REQUEST["view"]) || isset($_REQUEST['keyword'])) && isset($search_votes) && $search_votes === true) {
+      if((isset($_REQUEST["view"]) || isset($_REQUEST['keyword'])) && empty($search_votes)) {
          $keyword = isset($_REQUEST['keyword']) ? '&keyword=' . urlencode($_REQUEST['keyword']) : '';
-        // Setting type is here only to avoid notice or more conditions.
-        // It won't cause to show last notes (type=0) as $_REQUEST['keyword'] presence has higher priority.
-        $type = 0;
          echo "<p><a href=\"?view=notes&page=$page&type=$type$keyword\">Next 10</a>";
-      } elseif (isset($_REQUEST["view"]) && isset($search_votes) === false) {
+      } elseif (isset($_REQUEST["view"]) && !empty($search_votes)) {
          echo "<p>";
          if (isset($NextPage) && $NextPage > 1) {
            echo "<a href=\"?view=notes&page=$PrevPage&type=$type{$isSearch}\">&lt; Prev 25</a> ";


« previous php.webmaster (#28600) next »