Skip to content
This repository has been archived by the owner on Jul 22, 2022. It is now read-only.

Avoid "Notice: Undefined index: type" #34

Closed
wants to merge 1 commit into from
Closed

Avoid "Notice: Undefined index: type" #34

wants to merge 1 commit into from

Conversation

StephanePate
Copy link
Contributor

@StephanePate StephanePate commented Jun 4, 2016

when "editing widgets" of a dashboard and trying to "add" in the backend, you get that notice because $parameters['type'] is undefined.

NB : it works in SonataPageBundle because BlockAdmin extends BaseBlockAdmin for which getPersistentParameters() sets the key "type" for $parameters

if (!$parameters['type']) in BlockAdminController (line 90) supposes that key "type" for parameters exists

if (!$parameters['type']) in BlockAdminController (line 90) supposes that key "type" for parameters exists
@core23
Copy link
Member

core23 commented Jun 5, 2016

Please add a little bit more information. When do you get this error?

@soullivaneuh
Copy link
Member

According to the new Sonata version management and next major release plan, this project has been refactored regarding branching and versioning.

If you see this message, your PR concerns a patch or a minor release and is not targeting the right branch.

So I'm closing this one, but don't see it as a refusal. If you think your work is still relevant and want to continue, feel free to reopen it on the right branch (e.g. the default one).

Regards.

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

Successfully merging this pull request may close these issues.

None yet

4 participants