StackOverflow2013

Note that there are some explanatory texts on larger screens.

plurals

POCakePHP: Proper way to delete a record securely
primarykey
Id
11113333
data
AcceptedAnswerId
11113574
AnswerCount
2
ClosedDate
CommentCount
0
CommunityOwnedDate
CreationDate
2012-06-20T05:29:51.103
FavoriteCount
0
LastActivityDate
2012-06-20T06:35:10.360
LastEditDate
2012-06-20T05:39:59.327
LastEditorUserId
66767
OwnerUserId
66767
ParentId
0
PostTypeId
1
Score
0
ViewCount
3348
LastEditorDisplayName
text
Body
<p>I have this in my Controller.</p> <pre><code>public function delete($id) { if($this->request->is('get')) { throw new MethodNotAllowedException(); } $this->Memberlist->id = $id; if (!$this->Memberlist->exists()) { throw new NotFoundException(__('Invalid list.')); } if ($this->Memberlist->delete()) { $this->Session->setFlash(__('List deleted.'), 'success'); return $this->redirect(array('action'=>'index')); } $this->Session->setFlash(__('List was not deleted.'), 'error'); return $this->redirect(array('action'=>'index')); } </code></pre> <p>My Model looks like this: (belongsTo)</p> <pre><code><?php class Memberlist extends AppModel { public $name = 'Memberlist'; public $belongsTo = array( 'Account' => array( 'className' => 'Account', 'foreignKey' => 'account_id' ) ); </code></pre> <p>In one of my views, I have something like this:</p> <pre><code>echo $this->Form->postLink('Delete', array('action' => 'delete', $list['Memberlist']['id']), array('class'=>'btn-mini btn', 'confirm' => 'Are you sure?')); </code></pre> <p>Which creates a HTML like this:</p> <pre><code><form id="post_4fe15efc0d284" method="post" style="display:none;" name="post_4fe15efc0d284" action="/Grid/memberlists/delete/9"> <input type="hidden" value="POST" name="_method"> <input id="Token1627936788" type="hidden" value="8756f7ad21f3ab93dd6fb9a4861e3aed4496f3f9" name="data[_Token][key]"> <div style="display:none;"> </form> <a class="btn-mini btn" onclick="if (confirm('Are you sure?')) { document.post_4fe15efc0d284.submit(); } event.returnValue = false; return false;" href="#">Delete</a> </code></pre> <p>The problem is that when I update the <code>ID</code> found in <code>action="/Grid/memberlists/delete/9"</code> using Firebug (or any developer tool), I can pretty much delete anything! Even from a different account. Even though I have the Security Component turned on.</p> <p>What would be the proper way to do this? I am thinking of checking the <code>account_id</code> against the account_id of the currently logged-in user. But I am just curious if CakePHP has something out-of-the-box that fixes this issue?</p>
Tags
<php><cakephp><cakephp-2.0><cakephp-2.1>
Title
CakePHP: Proper way to delete a record securely
singulars
PostAcceptedAnswerId
1. PO
  singulars
  PostTypePostTypeId
  PTAnswer
PostParentId
1. This table or related slice is empty.
PostTypePostTypeId
1. PTQuestion
UserLastEditorUserId
1. USwenbert
UserOwnerUserId
1. USwenbert
plurals
PostLinksPostIdRelatedPostId
1. This table or related slice is empty.
PostLinksRelatedPostIdPostId
1. This table or related slice is empty.
PostsAcceptedAnswerId
1. This table or related slice is empty.
PostsParentIdCreationDate
1. PO
  singulars
  PostTypePostTypeId
  PTAnswer
2. PO
  singulars
  PostTypePostTypeId
  PTAnswer
VotesPostIdCreationDate
1. This table or related slice is empty.
CommentsPostId
1. This table or related slice is empty.

Querying!

Guidance

A row detail

Detail views are divided into sections. All the information in the data section comes from columns in the selected row. The other sections display data from other, related rows.

Related data can be related in a to-one or a to-many fashion. Captions of data related in a to-many fashion link to a list view showing a filtered view of the table.

Try moving around until you find a non-empty to-many entry and click on the label to get to one. You can move back to the root by clicking on the database name in the header.