Enjoy an ad free experience by logging in. Not a member yet?
Register .
PM User | 16
New Coder
Join Date: Aug 2012
Posts: 65
Thanks: 5
Thanked 0 Times in 0 Posts
I am sorry to sound so ignorant about this, but I removed the ; at the end and nothing changed. I removed the while loop and the page just goes white. Im really not that good at this at all I rekon. Is there just an all around better way that I need to rewrite this code, sir?
PM User | 17
God Emperor
Join Date: Sep 2002
Location: Saskatoon, Saskatchewan
Posts: 15,752
Thanks: 4
Thanked 2,468 Times in 2,437 Posts
If the page goes white, then that's typically a syntactical error. You can check your logs or enable error reporting with
PHP Code:
ini_set ( 'display_errors' , 1 ); error_reporting ( E_ALL );
You can poll for the result count as well:
PHP Code:
printf ( 'There are %d records returned.' , sqlsrv_num_rows ( $_rs ));
Before that while loop. That needs to have at least one record. If it shows 0 (and no error), that indicates that your query simply has no matching results.
The semi-colon must not be on the while loop. Since the property of it takes the last resultant, that makes the returned fetch to become null (or false). Therefore it will never enter the conditional block to assign the variables individually.
PM User | 18
New Coder
Join Date: Aug 2012
Posts: 65
Thanks: 5
Thanked 0 Times in 0 Posts
Ah, ok. I now get the print out of:
There are 0 records returned.
I went to my SQL DB and ran ran a querry for ID number 249
(msag/modify2.php?StreetID=249) and the record is indeed in the DB but still nothing outputs to the screen but the empty form. the CSS and all the form boxes show, just empty. Since it shows the correct StreetID number in the URL im assuming that it is in fact finding the record, just wont show it, Its got to just be a simple syntax error on my part in tryin to convert it from MySQL to MsSQL, but i fail to see the error.
Any suggestions, sir?
PHP Code:
<?php include 'includes/head/head_main.php' ; ?>
<?php
( 'display_errors' , 1 );
error_reporting ( E_ALL );
'includes/header/header_main.php' ;
'includes/db/connect.php' ;
$_POST [ 'edit' ]))
$_StreetPrefix = $_rw [ 'StreetPrefix' ];
$_StreetName = $_rw [ 'StreetName' ];
$_LOWNUMBER = $_rw [ 'LOWNUMBER' ];
$_HighNumber = $_rw [ 'HighNumber' ];
$_EOB = $_rw [ 'EOB' ];
$_ESN = $_rw [ 'ESN' ];
$_COMMUNITY = $_rw [ 'COMMUNITY' ];
$_TELCO = $_rw [ 'TELCO' ];
$_MAP = $_rw [ 'MAP' ];
$_DateEntered = $_rw [ 'DateEntered' ];
$_DateModified = $_rw [ 'DateModified' ];
$_StreetID = $_rw [ 'StreetID' ];
$_SQL = "UPDATE MSAG SET StreetPrefix='$_StreetPrefix',
;
sqlsrv_query ( $_SQL ) or die( sqlsrv_error ());
//header("Location : index.php?id=$_id");
$_html = "<tabel><tr>
;
$_StreetID = (int)( $_GET [ 'StreetID' ]);
$_sql = "SELECT * FROM MSAG WHERE StreetID=$_StreetID" ;
$_rs = sqlsrv_query ( $conn , $_sql );
printf ( 'There are %d records returned.' , sqlsrv_num_rows ( $_rs ));
$_rw = sqlsrv_fetch_array ( $_rs ));
$_StreetID = $_rw [ 'StreetID' ];
$_StreetPrefix = $_rw [ 'StreetPrefix' ];
$_StreetName = $_rw [ 'StreetName' ];
$_LOWNUMBER = $_rw [ 'LOWNUMBER' ];
$_HighNumber = $_rw [ 'HighNumber' ];
$_EOB = $_rw [ 'EOB' ];
$_ESN = $_rw [ 'ESN' ];
$_COMMUNITY = $_rw [ 'COMMUNITY' ];
$_TELCO = $_rw [ 'TELCO' ];
$_MAP = $_rw [ 'MAP' ];
$_DateEntered = $_rw [ 'DateEntered' ];
$_DateModified = $_rw [ 'DateModified' ];
$_html = " <center>
;
?>
<?php echo $_html ; ?>
Last edited by willscarlet; 12-06-2012 at 04:23 PM ..
PM User | 19
God Emperor
Join Date: Sep 2002
Location: Saskatoon, Saskatchewan
Posts: 15,752
Thanks: 4
Thanked 2,468 Times in 2,437 Posts
This may not work:
StreetID='$_StreetID'. MySQL is one of the only databases that allows loose datatyping (and it is a feature of it). If StreetID is an integer type, you must remove the quotations around it. Same goes with any assignment in the SQL code.
I'm surprised the die doesn't indicate a datatype issue.
Where are these $_rw coming from:
PHP Code:
if(isset( $_POST [ 'edit' ])) $_StreetPrefix = $_rw [ 'StreetPrefix' ]; $_StreetName = $_rw [ 'StreetName' ]; $_LOWNUMBER = $_rw [ 'LOWNUMBER' ]; $_HighNumber = $_rw [ 'HighNumber' ]; $_EOB = $_rw [ 'EOB' ]; $_ESN = $_rw [ 'ESN' ]; $_COMMUNITY = $_rw [ 'COMMUNITY' ]; $_TELCO = $_rw [ 'TELCO' ]; $_MAP = $_rw [ 'MAP' ]; $_DateEntered = $_rw [ 'DateEntered' ]; $_DateModified = $_rw [ 'DateModified' ]; $_StreetID = $_rw [ 'StreetID' ];
?
I'd suspect you intend to use $_POST for those.
PM User | 20
New Coder
Join Date: Aug 2012
Posts: 65
Thanks: 5
Thanked 0 Times in 0 Posts
The $_rw Variable is declared in the while loop.
while($_rw = sqlsrv_fetch_array($_rs));
Also, If I try and remove the while loop, if statement, or the update/delete feature (Just trying to get the output to work, then i'll worry about updating and deleting) the screen again just goes white.
The code below, inside the output form (value=\"$_StreetName\" ) if you remove the $_StreetName and put any random text there, the text will show up, so it is not actually pulling the infor from the DB.
I striped the code down to just one field (this was just getting too complicated and jumbled up):
PHP Code:
<?php include 'includes/head/head_main.php' ; ?>
<?php
( 'display_errors' , 1 );
error_reporting ( E_ALL );
'includes/header/header_main.php' ;
'includes/db/connect.php' ;
$_POST [ 'edit' ]))
$_StreetName = $_POST [ 'StreetName' ];
$_StreetID = $_POST [ 'StreetID' ];
$_SQL = "UPDATE MSAG SET StreetName='$_StreetName'
;
sqlsrv_query ( $_SQL ) or die( sqlsrv_error ());
//header("Location : index.php?id=$_id");
$_html = "<tabel><tr>
;
$_StreetID = (int)( $_GET [ 'StreetID' ]);
$_sql = "SELECT * FROM MSAG WHERE StreetID=$_StreetID" ;
$_rs = sqlsrv_query ( $conn , $_sql );
printf ( 'There are %d records returned.' , sqlsrv_num_rows ( $_rs ));
$_rw = sqlsrv_fetch_array ( $_rs ));
$_StreetID = $_rw [ 'StreetID' ];
$_StreetName = $_rw [ 'StreetName' ];
$_html = " <center>
;
?>
<?php echo $_html ; ?>
Last edited by willscarlet; 12-06-2012 at 06:21 PM ..
PM User | 21
New Coder
Join Date: Aug 2012
Posts: 65
Thanks: 5
Thanked 0 Times in 0 Posts
in case anyone has a need for a smiliar script, This is the fixed code so far:
PHP Code:
<?php include 'includes/head/head_main.php' ; ?>
<?php
( 'display_errors' , 1 );
error_reporting ( E_ALL );
'includes/header/header_main.php' ;
'includes/db/connect.php' ;
$_POST [ 'edit' ]))
$_StreetName = $_POST [ 'StreetName' ];
$_StreetID = $_POST [ 'StreetID' ];
$_SQL = "UPDATE MSAG SET StreetName='$StreetName'
;
sqlsrv_query ( $_SQL ) or die( sqlsrv_error ());
$_html = "<tabel><tr>
;
$StreetID = (int)( $_GET [ 'StreetID' ]);
$query = "SELECT * FROM MSAG WHERE StreetID=$StreetID" ;
$resource = sqlsrv_query ( $conn , $query );
$result = sqlsrv_fetch_array ( $resource ))
"
. $result [ 'StreetPrefix' ]. "> <br></td>
. $result [ 'StreetName' ]. "> <br></td>
. $result [ 'StreetSuffix' ]. "> <br></td>
. $result [ 'StreetPostDir' ]. "> <br></td>
. $result [ 'COMMUNITY' ]. "> <br></td>
. $result [ 'LOWNUMBER' ]. "> <br></td>
. $result [ 'HighNumber' ]. "> <br></td>
. $result [ 'EOB' ]. "> <br></td>
. $result [ 'ESN' ]. "> <br></td>
. $result [ 'TELCO' ]. "> <br></td>
. $result [ 'MAP' ]. "> <br></td>
. $result [ 'ZONE' ]. "> <br></td>
. $result [ 'PostOffice' ]. "> <br></td>
.(( $result [ 'DateEntered' ] instanceof DateTime ) ? $result [ 'DateEntered' ]-> format ( 'Y-m-d' ) : $result [ 'DateEntered' ]). "> <br></td>
.(( $result [ 'DateUpdated' ] instanceof DateTime ) ? $result [ 'DateUpdated' ]-> format ( 'Y-m-d' ) : $result [ 'DateUpdated' ]). "> <br></td>
. $result [ 'Commments' ]. "> <br></td>
;
?>
Now all is needed is to make it able to update the record or delete the record. If anyone has done this in MsSQL, I owuld appreciate some guidance and advice. I will post the finished code once I figure it out though. and thank you guys for all your help so far
PM User | 22
God Emperor
Join Date: Sep 2002
Location: Saskatoon, Saskatchewan
Posts: 15,752
Thanks: 4
Thanked 2,468 Times in 2,437 Posts
Simply use elseif's to keep going with the if/else chain you have here. So you'd have an else if (isset($_POST['delete'])) block. Simply follow the same logic you have for the editing. You'll probably want to keep adding with the logic to the edit as well, otherwise the form only deals with one field modification.
PM User | 23
New Coder
Join Date: Aug 2012
Posts: 65
Thanks: 5
Thanked 0 Times in 0 Posts
I have the edit isset in there for it to update the record but im getting the error:
Notice: Undefined index: StreetID in C:\Inetpub\wwwroot\msag\modify2.php on line 18
My code is:
PHP Code:
<?php include 'includes/head/head_main.php' ; ?>
<?php
( 'display_errors' , 1 );
error_reporting ( E_ALL );
'includes/header/header_main.php' ;
'includes/db/connect.php' ;
$_POST [ 'edit' ]))
$StreetName = $_POST [ 'StreetName' ];
$StreetID = $_POST [ 'StreetID' ];
$_SQL = "UPDATE MSAG SET StreetName='$StreetName'
;
sqlsrv_query ( $conn , $_SQL ) or die( sqlsrv_error ());
$_html = "<tabel><tr>
;
$StreetID = (int)( $_GET [ 'StreetID' ]);
$query = "SELECT * FROM MSAG WHERE StreetID=$StreetID" ;
$resource = sqlsrv_query ( $conn , $query );
$result = sqlsrv_fetch_array ( $resource ))
"
. $result [ 'StreetPrefix' ]. "> <br></td>
. $result [ 'StreetName' ]. "> <br></td>
. $result [ 'StreetSuffix' ]. "> <br></td>
. $result [ 'StreetPostDir' ]. "> <br></td>
. $result [ 'COMMUNITY' ]. "> <br></td>
. $result [ 'LOWNUMBER' ]. "> <br></td>
. $result [ 'HighNumber' ]. "> <br></td>
. $result [ 'EOB' ]. "> <br></td>
. $result [ 'ESN' ]. "> <br></td>
. $result [ 'TELCO' ]. "> <br></td>
. $result [ 'PostOffice' ]. "> <br></td>
. $result [ 'ZONE' ]. "> <br></td>
. $result [ 'MAP' ]. "> <br></td>
.(( $result [ 'DateEntered' ] instanceof DateTime ) ? $result [ 'DateEntered' ]-> format ( 'Y-m-d' ) : $result [ 'DateEntered' ]). "> <br></td>
.(( $result [ 'DateUpdated' ] instanceof DateTime ) ? $result [ 'DateUpdated' ]-> format ( 'Y-m-d' ) : $result [ 'DateUpdated' ]). "> <br></td>
. $result [ 'Commments' ]. "> <br></td>
;
?>
what would you advice instead of mysql_real_escape_string.?
as we have discovered, this:
PHP Code:
$StreetName = mysql_real_escape_string ( $_POST [ 'StreetName' ]);
is not accepted
PM User | 24
God Emperor
Join Date: Sep 2002
Location: Saskatoon, Saskatchewan
Posts: 15,752
Thanks: 4
Thanked 2,468 Times in 2,437 Posts
Of course not, that's for use with mysql which requires an open connection to a mysql server. It won't work for SQLServer as they have different rules on escaping.
I'd recommend you bind. I guess you can probably force the data via hex or binary though as an alternative, but I'd suggest not reinventing the wheel with it. The prepared statements can accept a bind, and the sqlsrv_query itself can also accept an array to bind.
See the first example here for an example of the bind:
http://ca2.php.net/manual/en/function.sqlsrv-query.php you simply replace the data with the ? for a placeholder, then provide it as an array for the third parameter to the sqlsrv_query method.
As for this error, streetID is not a valid input name. It appears you will have an id method that is hidden, but it must be rearranged on the form to appear before the form submit. You also won't need to redeclare it for the delete, as long as its a part of the same form the delete will be able to access the same information when submitted. For the deletion though, it may be a better idea to force an "Are you sure you want to delete. . ." message, then when they submit that one send it off to another request with like doDelete or something for its values.
Jump To Top of Thread
Thread Tools
Rate This Thread
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts
HTML code is Off
All times are GMT +1. The time now is 11:31 AM .
Advertisement
Log in to turn off these ads.
Before you post, read our: 
