PHP - Only Showing 1 User

Actually, what i want to do is to use the email to fetch the $email,$password and $randomnumber from database after

Hi guys,

I am trying to put together a little system that allows users to log onto my website and access there own personal page. I am creating each page myself and uploading content specific to them which cannot be viewed by anyone else.

I have got the system to work up as far as:

1/ The user logs in
2/ Once logged in they are re-directed to their own page using 'theirusername.php'

Thats all good and working how I need it too. The problem I have is this. If I log onto the website using USER A details - I get taken to USER A's page like I should but - If I then go to my browser and type in USERBdetails.php I can then access USER B's page.

This cannot happen!! I need for USER A not to be able to access USER B profile - there is obviously no point in the login otherwise! If you are not logged in you obviously cannot access any secure page. That much is working!

Please find below the code I am using:


LOGIN

<?php
session_start();


function dbconnect()
{
    $link = mysql_connect("localhost", "username", "password") or die ("Error: ".mysql_error());

}
?>

<?php
if(isset($_SESSION['loggedin']))
{
    header("Location:" . strtolower($username) . ".php");
if(isset($_POST['submit']))
{
   $username = mysql_real_escape_string($_POST['username']);
   $password = mysql_real_escape_string($_POST['password']);
   $mysql = mysql_query("SELECT * FROM clients WHERE username = '{$username}' AND password = '{$password}'");
   if(mysql_num_rows($mysql) < 1)
   {
     die("Password or Username incorrect! Please <a href='login.php'>click here</a> to try again");
   }   $_SESSION['loggedin'] = "YES";
   $_SESSION['username'] = $username;
 $_SESSION['name']
   header("Location:" . strtolower($username) . ".php");
}
?> 

HEADER ON EACH PHP PAGE

<?php
session_start();
if(!isset($_SESSION['loggedin']))
{
    die(Access to this page is restricted without a valid username and password);
?>

---------------------------------------------------

Am I right in thinking it is something to do with the "loggedin" part?

The system I have here is adapted from a normal login system I have been using for years. The original just checks the details and then does a 'session start'. This one obviously has to re-direct to a user specific page. To do this I used the <<header("Location:" . strtolower($username) . ".php");>> line to redirect to a page such as "usera.php" or "userb.php"

Any help would be greatly appreciated! 

Ta

Hi,

I am getting frustrated beyond belief at the moment with trying to get a very simple script to run, I am using PHP 5.3.3 and MySQL 5.1 on a Win2k8 server with IIS7.5. Basically my script is connecting to a local database, running a single select query, returning those rows and building up a string from them.

The problem is that I am receiving complete BS responses from PHP that the access is denied for the user being specified. This is complete rubbish since the user can connect via mysql, sqlyog, ASP.NET MVC without issue but for some bizarre reason it is not working via PHP.

The code for the script is here :

Code: [Select]
<?php

$mysql = mysql_connect('127.0.0.1:3306', 'myuser', 'mypass', 'mydatabase');


if (!$mysql) {
  die(mysql_error());
  $content = "<nobr></nobr>";
} else {
  
  $result = mysql_query('SELECT * FROM tblEventGroup'); 
  $content = "<nobr>";
  
  if ($result) {     
      while($row = mysql_fetch_assoc($result)) {
       $content .= "<span>";
       $content .= $row['GroupName'];
       $content .= "</span>";
       $content .= "<a href=\"../Event/EventSearch?groupid=";
       $content .= $row['GroupId'];
       $content .= "\" target=\"_blank\">Book here</a> ";
      }
  }
  
  mysql_close($mysql);
  $content .= "</nobr>";

}

?>
I cannot for the life of me understand what the problem is, the return error is

Access denied for user 'myuser'@'localhost' (using password: YES)

Hallo everybody, i have the following code. but i get allways this error while the user exist in the database. User not found! what do i do wrong?   thank you very much for your help Rafal

<html>
<head>
<?php
$connection = mysql_connect("db.xyz.com", "username", "password")
or die ("connection fehler");
mysql_select_db("db0123456789")
or die ("database fehler");
$email = $_POST["inp_email"];
$pwd = $_POST["inp_pwd"];
if($email && $pwd)
{
$chkuser = mysql_query("SELECT email FROM gbook WHERE email = '($email)' ");
$chkuserare = mysql_num_rows($chkuser);
echo $email;
echo $pwd;
if ($chkuserare !=0)
{
$chkpwd = mysql_query("SELECT pwd FROM gbook WHERE email = '($email)' ");
$pwddb = mysql_fetch_assoc($chkpwd);
if ($pwd != $pwddb["pwd"])
{
echo "password is wrong!";
}
else
{
echo "login successed";
}
}
else
{
echo "User not found!";
}
}
else
{
echo "Pleas enter your email and password!";
}
mysql_close($connection);
?>
</head>
<body>
<form action="login.php" method="post">
Email <input type="text" name="inp_email"><br>
Password <input type="text" name="inp_pwd"><br>
<input type="submit" name="submit" value="login">
</form>
</body>
</html>

Edited by rafal, 21 September 2014 - 04:33 PM.

Hello, i've got some shop script which has 2 payment modules which i'd like to use for something else, the payment modules only work if the user is logged in though, i tried to make them standalone scripts but that didn't work out too well.

So now i decided to go another way and just let everyone have the same session so everyone will be using the same username&password automatically.

the index file looks like this:
Code: [Select]
<?php
include('./inc/config.php');
include('./inc/functions.php');
include('./lang/'.$language.'.lng');
$id = addslashes($_REQUEST["id"]);
$user = addslashes($_REQUEST["username"]);
$pass = addslashes($_REQUEST["password"]);
$language = strtolower($language);
if(empty($id)) $id =1;
$file = mysql_query('SELECT * FROM navi_'.$language.' WHERE id="'.$id.'"');
if(mysql_num_rows($file)>0)
$file = mysql_fetch_array($file);
else
$file = mysql_fetch_array(mysql_query('SELECT * FROM navi_'.$language.' WHERE id="404"'));

if(!empty($user) AND !empty($pass))
{$query = mysql_query('SELECT * FROM users WHERE username="'.$user.'" AND pass="'.md6($pass).'"');
if(mysql_num_rows($query) == 1) {$_SESSION[$session_prefix."user"] = ucfirst($user); echo'<meta http-equiv="refresh" content="0; url=index.php?id=8">';}
else $error = 'Username oder Passwort ist falsch.';}

include('./designe/'.$designe.'/head.tpl');
include('./designe/'.$designe.'/navi.php');
include('./designe/'.$designe.'/middle.tpl');

if(file_exists('./pages/'.$file["file"]))
{echo'<h1>'.ucfirst($file["title"]).'</h1>';
include('./pages/'.$file["file"]);}
if(!empty($error)) echo '<font color="red">'.$error.'</font>'; 

include('./designe/'.$designe.'/foot.tpl');
?>

Now i tried alot of things including adding:
Code: [Select]
session_start();
$_SESSION["username"] = "peter";
$_SESSION["user"] = "peter";
$_SESSION["id"] = "1";
$_SESSION["pass"] = "peter";
$_SESSION["password"] = "peter";
or
Code: [Select]
$id = "1";
$user = "peter";
$username = "peter";
$pass = "peter";
$password = "peter";
also a combination of both, nothing works, but i don't understand why ? Any help is appreciated.

/Edit, i tried adding it to the paymentmodule .php aswell, but no luck.

Hallo everybody, the user is in the table, but i get error (user not found!).   thank you very much for your help Rafal

<!DOCTYPE html>
<html>
<head>
<title>index</title>
<meta http-EQUIV="CONTENT-LANGUAGE" content="en">
<?php
SESSION_START();
include("abc.php");
$link2 = mysqli_connect("$hoster", "$nameuser", "$password", "$basedata")
or die ("connection error" . mysqli_error($link2));
$email = $_POST["inp_email"];
$pwd = $_POST["inp_pwd"];
if($email && $pwd)
{
$chkuser = mysqli_query("SELECT email FROM $table2 WHERE email = '$email' ");
$chkuserare = mysqli_num_rows($chkuser);
if ($chkuserare !=0)
{
$chkpwd = mysqli_query("SELECT pwd FROM $table2 WHERE email = '$email'");
$pwddb = mysqli_fetch_assoc($chkpwd);
if (md5($pwd) != $pwddb["pwd"])
{
echo "Password is wrong!";
}
else
{
$_SESSION['username'] = $email;
header ('Location:list.php');
}
}
else
{
echo "user not found!";
}
}
else
{
echo "enter your Email and Password!";
}
mysqli_close($link2);
?>
</head>
<body style="font-family: arial;margin: 10; padding: 0" bgcolor="silver">
<font color="black">
<br>
<form action="index.php" method="post">
<b>Login</b><br><br>
<table width="100%">
<tr><td>
Email:<br><input type="text" name="inp_email" style="width:98%; padding: 4px;"><br>
Password:<br><input type="password" name="inp_pwd" style="width:98%; padding: 4px;"><br>
<br>
<input type="submit" name="submit" value="Login" style="width:100%; padding: 4px;">
</td></tr>
</table>
</form>
</font>
</body>
</html>

hello. I need your help please. I'm building logistics website with user panel and admin panel. I've done all login and register forms. now I want to : admin can add package with: tracking number , weight , cost , and declaration form. user can fill declaration form after admin add package to user panel. then admin can see the declared form. is it possible in php? thank you in advance

I decided to redo the individual roster page I have on my site and came up with this.


<?php
if(isset($_GET['username']) && $_GET['username'] != '')
{
$username = $_GET['username'];
$query = "SELECT bio.username AS username,  bio.charactername AS charactername, bio.id AS id, bio.style_id AS style FROM `efed_bio` AS bio ON bio.id = ebw.bio_id WHERE bio.username = '$username'";
if(!$result = mysql_query($query))
while ($row = mysql_fetch_assoc($result))
{
$fieldarray=array('id','username','charactername','style');
foreach ($fieldarray as $fieldlabel)
{
if (isset($row[$fieldlabel]))
{ 
$$fieldlabel=$row[$fieldlabel];
$$fieldlabel=cleanquerydata($$fieldlabel);
}
}
}
}
?>

        <h1><?php echo $charactername; ?>'s Biography</h1>
<?php echo getBioMenu($style,$username); ?>

<?
//If the GET page variable is biography, display the biography page
if ($page == 'biography')
{
echo getBiopgrahy($style,$id);
}
//Or if the GET page variable is gallery, display the gallery page
elseif ($page == 'gallery')

{
echo getGallery($style,$id);
}
//If the GET page variable is news, display the news page
elseif ($page == 'news')

{
echo getNews($$id);
}
//If the GET page variable is segments, display the segments page
elseif ($page == 'segments')

{
echo getBioSegments($id,S);
}
//If the GET page variable is matches, display the matches page
elseif ($page == 'matches')

{
echo getBioSegments($id,M);
}
elseif ($page == 'chronology')

{
echo getChronology($id);
}
//Otherwise display the bio
else
{
echo getBio($style,$id);
}

?>




The list of characters are listed he

http://kansasoutlawwrestling.com/roster

An example of someone's individual roster page is located he

http://kansasoutlawwrestling.com/bio?username=oriel

hello team,

can you please look at this code snippet and tell me why it is showing the following data this way?

<?php
#  jeff Exp $
include("app.php");

$page->set("title", "Business & Commerical");

$what = $page->getvar("what");
$type = $page->getvar("type");

$dt = new xDataTable('width="500"');
$form = new Form();

if($what == "submit") $html->set("readonly", true);

$dt->header($_types{$type} . " Quote");

$form->get_contact_form(true);


When I run it, it shows like this:

set("title", "Business & Commerical"); $what = $page->getvar("what"); $type = $page->getvar("type"); $dt = new xDataTable('width="500"'); $form = new Form(); if($what == "submit") $html->set("readonly", true); $dt->header($_types{$type} . " Quote"); $form->get_contact_form(true); switch ($type) { case "comp": $dt->left("Business Type:"); $dt->cell($html->select("business_type", $_business_type, 1, "", $page->getvar("business_type"), "(select)")); $dt->left("Federal Tax ID:"); $dt->cell($html->textfield("tax_id", $page->getvar("tax_id"), 12)); $dt->left("If Individual, Owner SSN#:"); $dt->cell($html->textfield("ssn", $page->getvar("ssn"), 12)); $dt->cell("Spouse SSN#:"); $dt->cell($html->textfield("spouse_ssn", $page->getvar("spouse_ssn"), 12));

just a small sample.

What am I doing wrong?

Thanks alot in advance

Hi,

I am using the include function to link my header and menu up to my files.

My folder structure is:

WWW Folder
    -head.html
    -menu.html
    -homepage.php
Images Folder
Blog Folder
    -index.php

The head.html and menu.html have links to images kept within the images folder.
homepage.php and index.php both have include functions to include the menu and header thus including the images.

homepage.php works fine the images show up however blog.php links to the head and menu but does not show up the images.

my include function for index.php is:
<?php
include ("..\menu.html");
?>

I have attached a print screen of what the index.php looks like, its obvious that its picking up the head and menu as its showing the image names but not showing the images.

Can anybody help it's really confusing me? Thanks

Hi Friends,

I have form in that i am sending some values using POST method. But i cant able to get values using $_POST['var'] or $_REQEUST['var'] Data is not coming in IE6,

But it is coming in firefox and other browsers even in IE8.

What may be problem? any settings i have to make for IE6. ?

Any suggestions

Hi all,
I have table, named: tb_event (id, f_date, f_title, f_image, f_status)
How to manage, while if 'f_status' from all records NOT EQUAL to 'Published' then show ABCDE
Note: on tb_event should be only the latest record (1 record only) that will be shown/published

I have code:

<?
$qry = mysql_query(
"SELECT * FROM tb_event WHERE f_status = 'Published'") or die(mysql_error());
while($res = mysql_fetch_array($qry))
{
if ($res[f_status] = "Published") {
   echo '
   <img src="images/uploads/'.$res[f_image].'" border="0" />
   ';
   }
   else
   {
   echo '
   ABCDE
   ';
   }
}
?>

The result is 'ABCDE' not shown while f_image (all records) is OFF 

Please help me.. thanks

hi all
I have a table which i have got results from a database. My question If i want to display the details of a particular row. How do I do that?

Thanks Rgds

Hey all, Been coding another script for my website which allows me to update a users Rank, allthough, when I try and view the page its giving me a White Blank Page. I carn't accually see whats wrong in my Script...


<?php
session_start();
include ("../includes/config.php");
include ("../includes/function.php");
logincheck();

ini_set ('display_errors', 1);
error_reporting (E_ALL);

$username = $_SESSION['username'];

// Owner Script - Change Users Rank

$userlevel = mysql_query ("SELECT * FROM users WHERE username = '$username' LIMIT 1") or die ("Error Finding Right Users " . mysql_error());
$levelright = mysql_fetch_object($userlevel);

// If userlevel isn't 5, move them on...
if ($levelright->userlevel == "1" || $levelright->userlevel == "2" || $levelright->userlevel == "3" || $levelright->userlevel == "4"){
header ("Location: 404.php");
die();
}

// Start Change Rank...
if (strip_tags($_POST['changerank'])){
$name = $_POST['name'];
$userchange = $_POST['userchange'];
$real = mysql_query ("SELECT * FROM users WHERE username = '$userchange' LIMIT 1");
$number = mysql_num_rows($real);
if ($number != "1"){
echo ("$userchange, is NOT registered! - Simple Words : No Such User!");
}
if ($name == 1){
$reprand = rand(100, 50000);
mysql_query ("UPDATE users SET rank = 'Learner Driver (1)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 1

if ($name == 2){
$reprand = rand(50000, 200000);
mysql_query ("UPDATE users SET rank = 'Passed Driver (2)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 2

if ($name == 3){
$reprand = rand(200000, 500000);
mysql_query ("UPDATE users SET rank = 'New Driver (3)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 3

if ($name == 4){
$reprand = rand(500000, 2500000);
mysql_query ("UPDATE users SET rank = 'Speeding Driver (4)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 4

if ($name == 5){
$reprand = rand(2500000, 7000000);
mysql_query ("UPDATE users SET rank = 'Skilled Driver (5)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 5

if ($name == 6){
$reprand = rand(7000000, 20000000);
mysql_query ("UPDATE users SET rank = 'Maniac Driver (6)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 6

if ($name == 7){
$reprand = rand(20000000, 45000000);
mysql_query ("UPDATE users SET rank = 'Wanted Racer (7)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 7

if ($name == 8){
$reprand = rand(45000000, 90000000);
mysql_query ("UPDATE users SET rank = 'Pro Driver (8)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 8

if ($name == 9){
$reprand = rand(90000000, 115000000);
mysql_query ("UPDATE users SET rank = 'Supreme Racer (9)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 9

if ($name == 10){
$reprand = rand(115000000, 150000000);
mysql_query ("UPDATE users SET rank = 'Super Supreme Racer (10)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 10

if ($name == 11){
$reprand = rand(150000000, 210000000);
mysql_query ("UPDATE users SET rank = 'Show Off Driver (11)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 11

if ($name == 12){
$reprand = rand(210000000, 280000000);
mysql_query ("UPDATE users SET rank = 'Extreme Show Off Driver (12)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 12

if ($name == 13){
$reprand = rand(280000000, 350000000);
mysql_query ("UPDATE users SET rank = 'Trained Driver (13)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 13

if ($name == 14){
$reprand = rand(350000000, 430000000);
mysql_query ("UPDATE users SET rank = 'Super Trained Driver (14)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 13

if ($name == 15){
$reprand = rand(430000000, 520000000);
mysql_query ("UPDATE users SET rank = 'Legendry Racer (15)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 15

if ($name == 16){
$reprand = rand(520000000, 600000000);
mysql_query ("UPDATE users SET rank = 'Most Legendry Racer (16)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 16

if ($name == 17){
$reprand = rand(600000000, 1000000000);
mysql_query ("UPDATE users SET rank = 'Official SD Legend (17)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 17

if ($name == 18){
$reprand = rand(1000000000, 1000000010);
mysql_query ("UPDATE users SET rank = 'Hidden Rank! (18)' AND rep = '$reprand' WHERE username = '$userchange'") or die ("Error Updating rank " . mysql_error());
} // Name 18

echo ("Successfully updated $userchange Rank!");
mysql_query("INSERT INTO `inbox` ( `id` , `to` , `from` , `message` , `date` , `read` , `saved` , `event_id` )
VALUES (
'', '$userchange', 'System', 'Your Rank has now been changed!', '$date', '0', '0', '0')") or die ("Error updating there inbox!" . mysql_error());

} // Rank Change

if (strip_tags($_POST['selectname'])){
$writenname = $_POST['nameinput'];
$rankinput = $_POST['rankinput'];
$repinput = $_POST['repinput'];
$findme = mysql_query ("SELECT * FROM users WHERE username = '$writenname' LIMIT 1") or die ("Error Searching Names " . mysql_error());
$found = mysql_num_rows($findme);
if ($found != "1"){
echo ("$writenname, isn't Registered! - Simple Terms : No Such User");
}else{
mysql_query ("UPDATE users SET rank = '$rankinput' AND rep = '$repinput' WHERE username= '$writenname'") or die ("Error Updating Rank + Rep " . mysql_error());
echo ("$writenname Rank has now been changed!");
mysql_query("INSERT INTO `inbox` ( `id` , `to` , `from` , `message` , `date` , `read` , `saved` , `event_id` )
VALUES (
'', '$writenname', 'System', 'Your Rank has now been changed!', '$date', '0', '0', '0')") or die ("Error updating there inbox!" . mysql_error());
} // Found?
} // Post selectname
?>
<html>
<head>
<title>Change Users Rank</title>
</head>
<body class='body'>
<form action='' method='POST' name='Change Rank'>
<table width='40%' border='1' cellpadding='0' cellspacing='0' class='table' align='center'>
<tr>
<td class='header' align='center' colspan='2'>Change Users Rank?</td>
</tr>
<tr>
<td class='omg' align='center'>This will change the users Current Rank!</td>
</tr>
<tr>
<td align='right'>Username:</td><td><input type='text' name='userchange' class='input'></td>
</tr>
<tr>
<td><select name='name' size='1' class='dropbox'>
<option value="1">Learner Driver (1)</option>
            <option value="2">Passed Driver (2)</option>
<option value="3">New Driver (3)</option>
<option value="4">Speeding Driver (4)</option>
<option value="5">Skilled Driver (5)</option>
<option value="6">Maniac Driver (6)</option>
<option value="7">Wanted Racer (7)</option>
<option value="8">Pro Driver (8)</option>
<option value="9">Supreme Racer (9)</option>
<option value="10">Super Supreme Racer (10)</option>
<option value="11">Show Off Driver (11)</option>
<option value="12">Extreme Show Off Driver (12)</option>
<option value="13">Trained Driver (13)</option>
<option value="14">Super Trained Driver (14)</option>
<option value="15">Legendry Racer (15)</option>
<option value="16">Most Legendry Racer (16)</option>
<option value="17">Official SD Legend (17)</option>
<option value="18">Hidden Rank! (18)</option></td>
</tr>
<tr>
<td class='omg' align='center' colspan='2'><input type='submit' name='changerank' class='button'></td>
</tr>
</table>
<br />
<table width='40%' border='1' cellpadding='0' cellspacing='0' class='table' align='center'>
<tr>
<td class='header' align='center' colspan='2'>Change Rank?</td>
</tr>
<tr>
<td class='omg' align='center'>This will change the <strong>posted</strong> users Rank and Rep!</td>
</tr>
<tr>
<td align='right'>Username:</td><td><input type='text' name='nameinput' class='input'></td>
</tr>
<tr>
<td align='right'>Rank:</td><td><input type='text' name='rankinput' class='input'></td>
</tr>
<tr>
<td align='right'>Rep:</td><td><input type='text' name='repinput' class='input'></td>
</tr>
<tr>
<td align='center' colspan='2'><input type='submit' name='selectname' class='button'></td>
</tr>
</table>
</form>
</body>
</html>


Anyone able to see where I'm going wrong?

Thanks

The following code should output 1743425391, but it does not show anything
<?php
$xml_feed_url = 'http://api.eve-online.com/eve/CharacterID.xml.aspx?names=TX1';
$ch = curl_init();
curl_setopt($ch, CURLOPT_URL, $xml_feed_url);
curl_setopt($ch, CURLOPT_HEADER, false);
curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
$xml = curl_exec($ch);
curl_close($ch);


$xmlr = new SimpleXMLElement($xml);

$charid = $xmlr->result->characterID;

echo $charid;
?>

The xml is:
Code: [Select]
<eveapi version="2">
<currentTime>2011-02-01 05:22:38</currentTime>
−
<result>
−
<rowset name="characters" key="characterID" columns="name,characterID">
<row name="TX1" characterID="1743425391"/>
</rowset>
</result>
<cachedUntil>2011-03-01 05:22:38</cachedUntil>
</eveapi>
Any ideas where I went wrong?