1)
Something that stands out is one of your links:

  if ($screen > 0)
  {
    print <a href=\"view.php?$screen=".( $screen+1)."\"><<Previous
Entries</a>   ";
  }

The '$' in '?$screen' is probably not helping, as that'll probably translate
to '?0=1' (or similar).

And, don't you want that to be ($screen - 1)??

2)
Depending on what version of PHP you're running.
You're assuming that $screen will equal $_GET['screen'].
Which, since 4.2.0 and with default settings, is a bad assumption.

  if ($screen != $_GET['screen']) $screen = $_GET['screen']; // or just use
the assignment.

There's a register_globals setting you can change (default is false):
  http://php.net/ini.core#ini.register-globals
But, changing it to true also has security implications:
  http://php.net/security.globals

Then again, you're also referencing a lot of variables that aren't defined
in your example.
So, it's hard to know what you may be doing that you just didn't post.

3)
You're basing $pages on the ceil([number of rows] / [rows per page]).
Problem with this, the LIMIT won't allow [number of rows] to ever be greater
than [rows per page].
So, $pages will never be anything besides 0 or 1.

You may want to try running a 2nd, similar query, without the LIMIT, using
COUNT(*).
(just in case: http://dev.mysql.com/doc/refman/5.0/en/counting-rows.html)

4)
You have an extra '}' after the while scope that doesn't line up with
anything.
Since you stated the page loads, I'm assuming that's just because it's a
snippet.


- Jon L.

On Thu, Apr 3, 2008 at 6:57 PM, Nasreen Laghari <[EMAIL PROTECTED]>
wrote:

> Hi,
>
> I'm trying to put limit of data fetch as well as dynamic navigation. Below
> is the try which I did but something is going wrong as $screen value is not
> increasing so when I click on NEXt button it refreshes the same page and
> Previous button does even show.
>
> Could you kindly have look on the coding and help me where i'm mistaking.
>
> Regards
>
> Nasreen
>
>
> <?php
> $rows_per_page = 10;
> $i=0;
> if (!isset($screen))
>  $screen=0;
> $start = $screen * $rows_per_page;
> $sql = "SELECT * FROM gig g, venue v WHERE g.gigName LIKE
> '%".$gig_name."%' OR g.gig_date LIKE '%".$sdate."%' OR g.genre LIKE
> '%".$genre."%' OR g.ticket_price LIKE '%".$ticket_price1."%' OR
> g.ticket_price LIKE '%".$ticket_price2."%' OR v.venueName LIKE
> '%".$vname."%' OR v.vCity LIKE '%".$city."%' order by gig_Date LIMIT
> $start,$rows_per_page";
> $result = mysql_query($sql) or die("Query error: ". mysql_error());
> $num_rows = mysql_num_rows($result) ;
> $pages = ceil($num_rows / $rows_per_page);
> $j=0;
> while ($row = mysql_fetch_array($result))
> {
>  global $limit;
>  $j = $j+1;
>      $gigid = $row['gigid'];
>   $gigname = $row['gigName'];
>   $sdate = $row['gig_fdate'];
>   $fdate =$row['gig_tdate'];
>   $genre = $row['genre'];
>   $ticket_price = $row['ticket_price'];
>   $gigdetail= $gigid;
>    echo("<br> $gigid <a href='detail.php?gigDetail=$gigid'>
>  $gigname</a>");
> }
>
> }
>
> if ($screen > 0)
>  {
>   print <a href=\"view.php?$screen=".( $screen+1)."\"><<Previous
> Entries</a>   ";
>  }
> else if ($screen < $pages)
>  {
>    $screen = $screen+1;
>  print "<a href=\"view.php?screen=".($screen)."\">Next Entries>></a>   ";
>  }
>
> ?>
>
>
>
>  
> ____________________________________________________________________________________
> You rock. That's why Blockbuster's offering you one month of Blockbuster
> Total Access, No Cost.
> http://tc.deals.yahoo.com/tc/blockbuster/text5.com
>

Reply via email to