Skip to content
This repository was archived by the owner on Aug 17, 2025. It is now read-only.

Update ResourceController.php#131

Closed
mschakulat wants to merge 1 commit into
phly:masterfrom
mschakulat:patch-1
Closed

Update ResourceController.php#131
mschakulat wants to merge 1 commit into
phly:masterfrom
mschakulat:patch-1

Conversation

@mschakulat
Copy link
Copy Markdown

Declaration of PhlyRestfully\ResourceController::deleteList() should be compatible with Zend\Mvc\Controller\AbstractRestfulController::deleteList($data)

Declaration of PhlyRestfully\ResourceController::deleteList() should be compatible with Zend\Mvc\Controller\AbstractRestfulController::deleteList($data)
@weierophinney
Copy link
Copy Markdown
Contributor

Accomplished with #135.

PHP allows reducing the number of required arguments in extending classes (doing so follows the LSP). As such, having no arguments, or an optional argument, for deleteList() is still valid. #135 updates the class to add $data = [].

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants