Edit function is INSERT instead of UPDATE CakePHP 2.1 - cakephp

I'm not exactly new to CakePHP though this is my first app in 2.x. I'm using the baked controller & view and am having a problem with the edit function.
For starters, here is what was baked:
public function edit($id = null) {
$this->User->id = $id;
if (!$this->User->exists()) {
throw new NotFoundException(__('Invalid user'));
}
if ($this->request->is('post') || $this->request->is('put')) {
if ($this->User->save($this->request->data)) {
$this->Session->setFlash(__('The user has been saved'));
$this->redirect(array('action' => 'index'));
} else {
$this->Session->setFlash(__('The user could not be saved. Please, try again.'));
}
} else {
$this->request->data = $this->User->read(null, $id);
}
}
When this is left alone
if (!$this->User->exists()) {
throw new NotFoundException(__('Invalid user'));
}
I'm told that I have an invalid user, even when I know without a doubt that the user does exist.
If I change it to this:
if (!$this->User->exists($id)) {
throw new NotFoundException(__('Invalid user'));
}
The correct edit form comes up, and is populated with the correct data, but then upon save it tries to INSERT instead of Update.
Typically this is because CakePHP doesn't have an ID to work with but there is a hidden ID field in the form and I have even tried to force the issue by putting the following before the save.
$this->request->data['User']['id'] = $id;
Any ideas what's going on here?

public function edit($id = null) {
if (!$this->User->exists($id)) {
throw new NotFoundException(__('Invalid user'));
}
if ($this->request->is('post') || $this->request->is('put')) {
$user_data = $this->User->findById($id);
if ($this->User->save($this->request->data)) {
$this->Session->setFlash(__('The user has been saved'), 'flash');
$this->redirect(array('action' => 'index'));
} else {
$this->Session->setFlash(__('The user could not be saved. Please, try again.'), 'flash');
}
} else {
$options = array('conditions' => array('User.' . $this->User->primaryKey => $id));
$this->request->data = $this->User->find('first', $options);
}
}

I got the same problem, code generated by bake, hidden id, etc..
Then I found that the field id on the database was not setted to autoincrement.
I just set to autoincrement and it works

Related

Failure to Update record CakePHP

I am currently trying to update my user before sending an email which will have a confirm link with a resetkey for the user to reset the password.
The problem is that the user is not updating whilst the key is being generated.
the code can be seen hereunder
public function forgotpassword() {
if ($this->request->is('post')) {
$mail = $this->request->data['ContactDetail']['email'];
$data = $this->User->find('first', array('conditions' => array('ContactDetail.email' => $mail)));
if (!$data) {
$message = __('No Such E-mail address registerd with us ');
$this->Session->setFlash($message);
} else {
$data['User']['resetkey'] = Security::hash(mt_rand(),'md5',true);
debug($data);
if ($this->User->saveAssociated($data)) {
$this->Session->setFlash(__('The user has been saved.'));
}
else {
$this->Session->setFlash(__('The user could not be updated. Please, try again.'));
}
$key = $data['User']['resetkey'];
$id = $data['User']['id'];
$mail = $data['ContactDetail']['email'];
$email = new CakeEmail('default');
$email->to($mail);
$email->from("xxxx#yahoo.com");
$email->emailFormat('html');
$email->subject('Password reset instructions from');
$email->viewVars(array('key' => $key, 'id' => $id, 'rand' => mt_rand()));
$email->template('reset');
if ($email->send('reset')) {
$message = __('Please check your email for reset instructions.');
$this->Session->setFlash($message);
} else {
$message = __('Something went wrong with activation mail. Please try later.');
$this->Session->setFlash($message);
}
}
$this->redirect('/');
}
}
I checked the execution log of MySQL but all I am seeing are selects no updates or even inserts.
This is probably something very dumb but I can't understand why it doesn't say anything.
Why do you use saveAssociated method for update single field?
Something like this would be easier:
$this->User->updateAll(
array( 'User.resetkey' => $hash ),
array( 'User.id' => $userId )
);
try using
$this->User->saveMany($data, array('deep' => true));
When saving. Also
if (empty($data)) {...
Might be a safer way to go
This will work for you-
$this->User->id = $data['User']['id'];
if ($this->User->save($data)) {
$this->Session->setFlash(__('The user has been saved.'));
}else {
$this->Session->setFlash(__('The user could not be updated. Please, try again.'));
}
Try to debug like:
if ($this->User->saveAssociated($data)) {
$this->Session->setFlash(__('The user has been saved.'));
}
else {
$this->Session->setFlash(__('The user could not be updated. Please, try again.'));
}
$log=$this->User->getDataSource()->getLog(false, false);
echo "<pre>";print_r($log);exit;

Changing single variable value in AppController

I have a Horse object that has a completed variable (text). I want to be able to click set complete in the View and then have that horse's completed variable set to "yes." After this, we would redirect the user to tasks/index. My thinking is to create setcomplete($id) function in Controller, so I could pass /setcomplete/4 and that horse's flag would change.
I have created a basic view for it, but the idea is that I would not need a view, but I do not know how to get around that....I was thinking of using $this->render() or something of this nature.
This is the basic code...pass in the id and then change the variable, then redirect to tasks/index.
I am not getting an error....its just not working, that's all.
public function setcomplete($id = null) {
if (!$this->Task->exists($id)) {
throw new NotFoundException(__('Invalid task'));
}
if ($this->request->is(array('post', 'put'))) {
$this->set('completed', 'yes');
if ($this->Task->save($this->request->data)) {
$this->Session->setFlash(__('The task has been updated.'));
return $this->redirect(array('action' => 'index'));
} else {
$this->Session->setFlash(__('The task could not be saved. Please, try again.'));
}
} else {
}
}
According to my understanding of your problem. you want to set the completed to yes for Task model.
i think you've done in a wrong way here.
what about the $this->request->data here?
Does it contain the data you want to save for the task model like the structure as below
Array(
'Task' => Array(
'id' => 4, // for eg..
'completed' => 'yes'
)
)
if not
you can solve the problem by doing like this. [Assuming you don't have $this->request->data]
public function setcomplete($id = null) {
if (!$this->Task->exists($id)) {
throw new NotFoundException(__('Invalid task'));
}
$this->Task->id = $id;
$this->Task->set('completed', 'yes')
if ($this->Task->save()) {
$this->Session->setFlash(__('The task has been updated.'));
return $this->redirect(array('action' => 'index'));
} else {
$this->Session->setFlash(__('The task could not be saved. Please, try again.'));
}
}

Get ID from table and update the row in CakePHP

I have Ipr_forms table in my Db i want to update the row but the following code give me job_id of that table.which is not my requirement.....i want table id to update the existing table
My controller code
$id =$this->request->data['IprForm']['id'];
$ipr_forms['job_id'] = $this->request->data['IprForm']['job_id'];
$ipr_forms['IprForm'] = $this->request->data['IprForm'];
$this->IprForm->id = $id;
//debug($id);
$ipr_forms_save = $this->IprForm->save($ipr_forms);
If i debug the id ,$id variable hold the job_id ....
here i can give you an example to edit any record in cakephp with multiple fields
just take look in function and you may have good idea
public function edit($id = null) {
if (!$id) {
throw new NotFoundException(__('Invalid post'));
}
$post = $this->Post->findById($id);
if (!$post) {
throw new NotFoundException(__('Invalid post'));
}
if ($this->request->is('post') || $this->request->is('put')) {
$this->Post->id = $id;
if ($this->Post->save($this->request->data)) {
$this->Session->setFlash(__('Your post has been updated.'));
$this->redirect(array('action' => 'index'));
} else {
$this->Session->setFlash(__('Unable to update your post.'));
}
}
if (!$this->request->data) {
$this->request->data = $post;
}
}
or you can also refer detail cakephp.org link with title of editing posts

how to delete records in cakephp

can anybody tell me how to delete records from the database?
I'm done with the rest part only deletion remains. I'm running the deletion command but
the data is not deleted from the database.Actuaaly i created an action in controller(delete_user) which contains the code for deleting the user from the database.
email value is stored in a session and i'm accessing it;s value using session read function
in the next controller.
function delete_user()
{
$email=$this->Session->read('User.email');
$this->User->delete($email, $cascade = true);
echo $this->Session->delete('User.email');
echo $this->Session->delete('User.username');
$this->redirect(array('action' => 'login'));
} );
i'm creating a link in home(View) page. and it is like this.
a href='/cakephp/cakephp/Users/delete_user'>Delete Profile</a>
i'm doing it right or wrong?
Please check the online documentation first before asking such a question.
Update 18/10/12
Try using CakePhp's magic find types:
$user = $this->User->findByEmail("bob#gmail.com");
$this->User->delete($user['User']['id']);
In your controller replace following code with delete action:
public function delete($id = null) {
if (!$this->request->is('post')) {
throw new MethodNotAllowedException();
}
$this->User->id = $id;
if (!$this->User->exists()) {
throw new NotFoundException(__('Invalid user'));
}
if ($this->User->delete()) {
$this->Session->setFlash(__('User deleted'));
$this->redirect(array('action' => 'index'));
}
$this->Session->setFlash(__('User was not deleted'));
$this->redirect(array('action' => 'index'));
}
and in ctp file add the following link:
<?php echo $this->Form->postLink(__('Delete'), array('action' => 'delete', ID OF THE RECORD, null, __('Are you sure you want to delete # %s?', ID OF THE RECORD)); ?>
then your problem will be solved...
you just need to pass the id as a first parameter:
$this->User->delete($id, $cascade);
with this you can delete it without cascade:
$this->Todo->delete($this->request->data('Todo.id'));
you can hook into this befor with
function beforeDelete(){
}
and here is my full delete action:
function delete($id = null){
$this->Todo->recursive = 1;
// new
$todo = $this->Todo->read(null,$id);
if($id == null || $todo == null) {
$this->Session->setFlash('Todo existiert nicht.');
$this->redirect(array('action' => 'index'));
return;
}
$this->set('todo',$todo);
$this->Todo->delete($this->request->data('Todo.id'));
$this->redirect(array('action' => 'index'));
return;
}

how I want the user id become static

I'm kind new to the cakephp and I want to know how to static the id where the id is on drop down or list. I have do hidden but its not enter the database. This is my coding:
This is in the controller
function add() {
if (!empty($this->data)) {
$this->Post->create();
if ($this->Post->save($this->data)) {
$this->Session->setFlash(__('The post has been saved', true));
$this->redirect(array('action' => 'index'));
} else {
$this->Session->setFlash(__('The post could not be saved. Please, try again.', true));
}
}
$users = $this->Post->User->find('list');
$this->set(compact('users'));
$this->set('userid',$this->Auth->user('id'));
}
this coding is in the ctp
<?php
echo $this->Form->input('user_id');
?>
The best way is to not send the user id to the browser and back at all, since that opens the possibility of form tinkering and security breaches/invalid results. Just inject the user id into the data before saving:
function add() {
if (!empty($this->data)) {
$this->Post->create();
// setting user id
$this->data['Post']['user_id'] = $this->Auth->user('id');
if ($this->Post->save($this->data)) {
$this->Session->setFlash(__('The post has been saved', true));
$this->redirect(array('action' => 'index'));
} else {
$this->Session->setFlash(__('The post could not be saved. Please, try again.', true));
}
}
$users = $this->Post->User->find('list');
$this->set(compact('users'));
}

Resources