Correction of errors #1
Conversation
| @@ -53,7 +53,7 @@ function nestedbox_core_entity_info() { | |||
| // but without the ID suffix. (In fact, you can set | |||
| // entity_operations_entity_uri() as your URI callback, which will use the | |||
| // value here). | |||
| 'path' => 'admin/content/nestedbox', | |||
| 'path' => 'entity_operations_entity_uri', | |||
There was a problem hiding this comment.
I'm afraid this is not how it works. This key is meant to contain a path, not a function name.
There was a problem hiding this comment.
You can look at EntityOperationsDefaultUIController::__construct() to see how the 'path' is used.
|
General comments: I am working on the thing locally. I rebased everything onto the 2259005-8-manychanges tag, which contains more fixes. And yes, nestedbox_type_form_delete_confirm() was missing. |
|
I will spend more time on this tomorrow. If you want to do something: Simply write some explanations for my questions. No need to do code at this moment. |
| @@ -77,6 +77,7 @@ function nestedbox_core_entity_info() { | |||
| 'label' => 'label', | |||
| ), | |||
| 'access callback' => 'nestedbox_type_access', | |||
| 'static cache' => TRUE, | |||
There was a problem hiding this comment.
According to the doc comment on hook_entity_info(), this is TRUE by default. So no need to put it here.. right?
There was a problem hiding this comment.
* - static cache: (used by DrupalDefaultEntityController) FALSE to disable
* static caching of entities during a page request. Defaults to TRUE.
|
Ok most of this is obsolete now, see https://github.com/donquixote/drupal-nestedbox/tree/7.x-1.x-2259005-15 |
Discovered and corrected some errors in 7.x-1.x...2259005... branch 7.x-1.x-donquixote-local
1 - as you yourself said "There is no more" Add nested box "button on the 'admin / content / nestedbox' page.", There was no "Add nested box", corrected.
2 - at removing such entity published a mistake, because there was no function nestedbox_type_form_delete_confirm, corrected.