0

我的控制器中有这个。

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'));
}

我的模型看起来像这样：（属于）

<?php

class Memberlist extends AppModel {
    public $name = 'Memberlist';
    public $belongsTo = array(
            'Account' => array(
            'className' => 'Account',
            'foreignKey' => 'account_id'
        )
    );

在我的一种观点中，我有这样的事情：

echo $this->Form->postLink('Delete', 
                    array('action' => 'delete', $list['Memberlist']['id']),
                    array('class'=>'btn-mini btn', 'confirm' => 'Are you sure?'));

它创建了一个这样的 HTML：

<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>

问题是，当我使用 Firebug（或任何开发人员工具）更新ID发现的内容时action="/Grid/memberlists/delete/9"，我几乎可以删除任何内容！即使来自不同的帐户。即使我打开了安全组件。

这样做的正确方法是什么？我正在考虑检查account_id当前登录用户的 account_id。但我只是好奇 CakePHP 是否有一些开箱即用的东西可以解决这个问题？

4

2 回答 2

3

您可以在模型中添加beforeDelete回调，并查询数据库并检查用户是否被允许删除记录，或者他是所有者。

于 2012-06-20T05:55:20.073 回答
2

要真正阻止您的用户完成不同的操作，例如删除不属于用户的内容，您应该使用Auth 组件。

我假设 Account 模型存储用户数据。您需要按照说明书中的教程进行操作，但我强调了删除权限将如何被拒绝。

您的 isAuthorized 方法看起来像这样：

public function isAuthorized($account) {
    // The owner of a post can edit and delete it
    if (in_array($this->action, array('edit', 'delete'))) {
        $memberListId = $this->request->params['pass'][0];
        if ($this->MemberList->isOwnedBy($memberListId, $account['id'])) {
            return true;
        }
    }

    // Default deny
    return false;
}

这将进入模型：

public function isOwnedBy($memberList, $account) {
    return $this->field('id', array('id' => $memberList, 'account_id' => $account)) === $post;
}
于 2012-06-20T06:35:10.360 回答