4 Replies Latest reply: Mar 8, 2012 7:41 PM by ggourde RSS

    Link: works as an a href, but not in php

    ggourde Community Member

      Upon submitting a form I want it to redirect to where it came from.

       

      Using this: <a href="form_update.php?beer_id=<?php echo $row_beer_update['beer_id']; ?>">

       

      I can create a link that will take me to the page I want with the right information. Unfortunately into won't work on the submit button. So I'd have to click once to submit the info, and again on the link to go to the page I want.

       

      Using this:

          

      mysql_select_db($database_beer_diary, $beer_diary);

        $Result1 = mysql_query($updateSQL, $beer_diary) or die(mysql_error());

        $row_beer_update = mysql_fetch_assoc($Result1);

         $updateGoTo = "detail_sheet.php?recordID=";

          if (isset($_SERVER['QUERY_STRING'])) {

            $updateGoTo .= (strpos($updateGoTo, '?')) ? "&" : "?";

            $updateGoTo .= $_SERVER['QUERY_STRING'];

          }

       

        header(sprintf("Location: %s", $updateGoTo));

      }

       

      It will go to the page I want, but it doesn't have the information I want on it.

       

      So trying to combine the  two:

       

      mysql_select_db($database_beer_diary, $beer_diary);

        $Result1 = mysql_query($updateSQL, $beer_diary) or die(mysql_error());

        $row_beer_update = mysql_fetch_assoc($Result1);

          $updateGoTo = "detail_sheet.php?recordID=".echo $row_beer_update['beer_id'];

         

      This is the closest I got (I've tried several variations, but none worked). On this attempt I keep on getting "parse eror" but I can't see what's missing.

       

      Any ideas?

      Thanks

        • 1. Re: Link: works as an a href, but not in php
          bregent CommunityMVP

          You are executing an update statement, so $Result1 contains only TRUE or FALSE; it is not a pointer to a recordset. Since you are updating a table, just use the primary key from your update's WHERE clause for the querystring value.

          • 2. Re: Link: works as an a href, but not in php
            ggourde Community Member

            Ah, bregent.  I see what you are saying about the $Result1.  So I tried this instead:

             

            if ((isset($_POST["MM_update"])) && ($_POST["MM_update"] == "form")) {

              $updateSQL = sprintf("UPDATE beer SET beer_name=%s, brewer=%s, style_id=%s, substyle_id=%s, country_id=%s, location_id=%s, container_id=%s, aroma_id=%s, aroma_notes=%s, appearance_id=%s, appearance_notes=%s, mouthfeel_id=%s, mouthfeel_notes=%s, flavor_id=%s, flavor_notes=%s, overall_id=%s, overall_notes=%s, recommend=%s, abv=%s, date_tasted_01=%s, image_id=%s WHERE beer_id=%s",

                                  

                                   GetSQLValueString($_POST['beer_id'], "int"));

             

             

              mysql_select_db($database_beer_diary, $beer_diary);

              $Result1 = mysql_query($updateSQL, $beer_diary) or die(mysql_error());

              $updateGoTo = "detail_sheet.php?recordID=".echo $updateSQL['beer_id'];

                if (isset($_SERVER['QUERY_STRING'])) {

                  $updateGoTo .= (strpos($updateGoTo, '?')) ? "&" : "?";

                  $updateGoTo .= $_SERVER['QUERY_STRING'];

                }

             

              header(sprintf("Location: %s", $updateGoTo));

            }

             

            But I am still getting the same parsing error.

            • 3. Re: Link: works as an a href, but not in php
              bregent CommunityMVP

              No, you don't want to append the entire SQL update statement to the querystring, you just want the ID.

               

                $updateGoTo = "detail_sheet.php?recordID=".$_POST['beer_id'];

               

              Also, I assume you are not showing us the entire sprintf() statement, as what you have posted would be invalid.

              • 4. Re: Link: works as an a href, but not in php
                ggourde Community Member

                Oh, I see!

                 

                Thank you very much (again) bregent.