Drupal 9 - PHP 8 - Modul funtkioniert nicht mehr
Eingetragen von Bavramor (129)am 02.05.2022 - 18:46 Uhr in
Liebe Community,
ich habe mein Drupal von 8 auf 9 geupdated und benutzte PHP 7.4. Jetzt habe ich den Zeitpunkt gleich dafür gewählt auf PHP 8 zu updaten. Läuft auch alles soweit, nur bei einem selbst gebauten Modul bekomme ich jetzt eine Fahlermeldung:
<?php
The website encountered an unexpected error. Please try again later.
Error: Attempt to assign property "format" on null in copy_fields_from_model_contract_node_edit_node_prepare_form() (line 57 of modules/custom/copy_fields_from_model_contract_node_edit/copy_fields_from_model_contract_node_edit.module).
?>Das betrifft genau diese Zeile;
<?php
$node->body->format = 'full_html';
?>Der Code meines Moduls ist zwar nicht elegant, hat aber seine Arbeit wie gewünscht getan. Im Prinzip kopiere ich aus einer anderen Node den Inhalt des Bodyfelds in ein Bodyfeld einer anderen Node. Der Code vom ganzen Modul sieht wie folgt aus:
<?php
<?php
use Drupal\node\NodeInterface;
use Drupal\Core\Form\FormStateInterface;
/**
* Implements hook_ENTITY_TYPE_prepare_form() for node entities.
*/
function copy_fields_from_model_contract_node_edit_node_prepare_form(NodeInterface $node, $operation, FormStateInterface $form_state)
{
//TODO Build NID as array and go through with loop
$arr_vertraege = array(6766, 6769, 6768, 6775, 6767, 6776, 6777, 6778, 6779, 6780, 6781, 6782);
foreach ($arr_vertraege as $mustervertraege) {
$nid = $mustervertraege;
$node_storage = \Drupal::entityTypeManager()->getStorage('node');
$node = $node_storage->load($nid);
if ($nid == $mustervertraege) {
$getcontract = $node->field_vertragsinhalt->value;
}
//TODO Works without ($operation == 'default')
if ($operation == 'edit') {
/*dpm("Modul an edit");*/
//TODO Shall always pull the current node and nid. But does not work yet with node creation. Error: Call to a member function id() on null in
$node = \Drupal::routeMatch()->getParameter('node');
$nid = $node->id();
$node_storage = \Drupal::entityTypeManager()->getStorage('node');
$node = $node_storage->load($nid);
if ($node->getType() == 'buch_anlegen') {
if ($node->field_vertrag_updaten->value == '1') {
// dpm($node->field_vertrag_updaten->value);
//var_dump($node);
if ($mustervertraege == $arr_vertraege[0]) {
$node->field_vertragsinhalt->value = $getcontract;
$node->body->format = 'full_html';
// don't save the node yet, it will be saved after submitting
}
}
if ($node->field_vertrag_updaten_2->value == '1') {
// dpm($node->field_vertrag_updaten->value);
//var_dump($node);
if ($mustervertraege == $arr_vertraege[1]) {
$node->field_vertragsinhalt_2->value = $getcontract;
$node->body->format = 'full_html';
// don't save the node yet, it will be saved after submitting
}
}
if ($node->field_vertrag_updaten_3->value == '1') {
// dpm($node->field_vertrag_updaten->value);
//var_dump($node);
if ($mustervertraege == $arr_vertraege[2]) {
$node->field_vertragsinhalt_3->value = $getcontract;
$node->body->format = 'full_html';
// don't save the node yet, it will be saved after submitting
}
}
if ($node->field_vertrag_updaten_4->value == '1') {
// dpm($node->field_vertrag_updaten->value);
//var_dump($node);
if ($mustervertraege == $arr_vertraege[3]) {
$node->field_vertragsinhalt_4->value = $getcontract;
$node->body->format = 'full_html';
// don't save the node yet, it will be saved after submitting
}
}
if ($node->field_vertrag_updaten_5->value == '1') {
// dpm($node->field_vertrag_updaten->value);
//var_dump($node);
if ($mustervertraege == $arr_vertraege[4]) {
$node->field_vertragsinhalt_5->value = $getcontract;
$node->body->format = 'full_html';
// don't save the node yet, it will be saved after submitting
}
}
if ($node->field_vertrag_updaten_6->value == '1') {
// dpm($node->field_vertrag_updaten->value);
//var_dump($node);
if ($mustervertraege == $arr_vertraege[5]) {
$node->field_vertragsinhalt_6->value = $getcontract;
$node->body->format = 'full_html';
// don't save the node yet, it will be saved after submitting
}
}
if ($node->field_vertrag_updaten_7->value == '1') {
// dpm($node->field_vertrag_updaten->value);
//var_dump($node);
if ($mustervertraege == $arr_vertraege[6]) {
$node->field_vertragsinhalt_7->value = $getcontract;
$node->body->format = 'full_html';
// don't save the node yet, it will be saved after submitting
}
}
if ($node->field_vertrag_updaten_8->value == '1') {
// dpm($node->field_vertrag_updaten->value);
//var_dump($node);
if ($mustervertraege == $arr_vertraege[7]) {
$node->field_vertragsinhalt_8->value = $getcontract;
$node->body->format = 'full_html';
// don't save the node yet, it will be saved after submitting
}
}
if ($node->field_vertrag_updaten_9->value == '1') {
// dpm($node->field_vertrag_updaten->value);
//var_dump($node);
if ($mustervertraege == $arr_vertraege[8]) {
$node->field_vertragsinhalt_9->value = $getcontract;
$node->body->format = 'full_html';
// don't save the node yet, it will be saved after submitting
}
}
if ($node->field_vertrag_updaten_10->value == '1') {
// dpm($node->field_vertrag_updaten->value);
//var_dump($node);
if ($mustervertraege == $arr_vertraege[9]) {
$node->field_vertragsinhalt_10->value = $getcontract;
$node->body->format = 'full_html';
// don't save the node yet, it will be saved after submitting
}
}
if ($node->field_vertrag_updaten_11->value == '1') {
// dpm($node->field_vertrag_updaten->value);
//var_dump($node);
if ($mustervertraege == $arr_vertraege[10]) {
$node->field_vertragsinhalt_11->value = $getcontract;
$node->body->format = 'full_html';
// don't save the node yet, it will be saved after submitting
}
}
if ($node->field_vertrag_updaten_12->value == '1') {
// dpm($node->field_vertrag_updaten->value);
//var_dump($node);
if ($mustervertraege == $arr_vertraege[11]) {
$node->field_vertragsinhalt_12->value = $getcontract;
$node->body->format = 'full_html';
// don't save the node yet, it will be saved after submitting
}
}
}
}
}
}
?>Hat jemand ein Tip für mich? Ich wäre darüer sehr dankbar.
Greetz Bavra
P.S.: Komplette Fehlermeldung aus den Logs:
<?php
Message Error: Attempt to assign property "format" on null in copy_fields_from_model_contract_node_edit_node_prepare_form() (line 57 of /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/modules/custom/copy_fields_from_model_contract_node_edit/copy_fields_from_model_contract_node_edit.module)
#0 [internal function]: copy_fields_from_model_contract_node_edit_node_prepare_form(Object(Drupal\node\Entity\Node), 'edit', Object(Drupal\Core\Form\FormState))
#1 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/lib/Drupal/Core/Entity/EntityForm.php(397): call_user_func_array('copy_fields_fro...', Array)
#2 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/lib/Drupal/Core/Entity/EntityForm.php(129): Drupal\Core\Entity\EntityForm->prepareInvokeAll('node_prepare_fo...', Object(Drupal\Core\Form\FormState))
#3 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/lib/Drupal/Core/Entity/ContentEntityForm.php(287): Drupal\Core\Entity\EntityForm->init(Object(Drupal\Core\Form\FormState))
#4 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/lib/Drupal/Core/Entity/EntityForm.php(96): Drupal\Core\Entity\ContentEntityForm->init(Object(Drupal\Core\Form\FormState))
#5 [internal function]: Drupal\Core\Entity\EntityForm->buildForm(Array, Object(Drupal\Core\Form\FormState))
#6 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/lib/Drupal/Core/Form/FormBuilder.php(531): call_user_func_array(Array, Array)
#7 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/lib/Drupal/Core/Form/FormBuilder.php(278): Drupal\Core\Form\FormBuilder->retrieveForm('node_buch_anleg...', Object(Drupal\Core\Form\FormState))
#8 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/lib/Drupal/Core/Controller/FormController.php(73): Drupal\Core\Form\FormBuilder->buildForm(Object(Drupal\node\NodeForm), Object(Drupal\Core\Form\FormState))
#9 [internal function]: Drupal\Core\Controller\FormController->getContentResult(Object(Symfony\Component\HttpFoundation\Request), Object(Drupal\Core\Routing\RouteMatch))
#10 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/lib/Drupal/Core/EventSubscriber/EarlyRenderingControllerWrapperSubscriber.php(123): call_user_func_array(Array, Array)
#11 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/lib/Drupal/Core/Render/Renderer.php(564): Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->Drupal\Core\EventSubscriber\{closure}()
#12 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/lib/Drupal/Core/EventSubscriber/EarlyRenderingControllerWrapperSubscriber.php(124): Drupal\Core\Render\Renderer->executeInRenderContext(Object(Drupal\Core\Render\RenderContext), Object(Closure))
#13 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/lib/Drupal/Core/EventSubscriber/EarlyRenderingControllerWrapperSubscriber.php(97): Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->wrapControllerExecutionInRenderContext(Array, Array)
#14 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/vendor/symfony/http-kernel/HttpKernel.php(158): Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->Drupal\Core\EventSubscriber\{closure}()
#15 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/vendor/symfony/http-kernel/HttpKernel.php(80): Symfony\Component\HttpKernel\HttpKernel->handleRaw(Object(Symfony\Component\HttpFoundation\Request), 1)
#16 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/modules/contrib/redirect_after_login/src/RedirectMiddleware.php(46): Symfony\Component\HttpKernel\HttpKernel->handle(Object(Symfony\Component\HttpFoundation\Request), 1, true)
#17 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/lib/Drupal/Core/StackMiddleware/Session.php(58): Drupal\redirect_after_login\RedirectMiddleware->handle(Object(Symfony\Component\HttpFoundation\Request), 1, true)
#18 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/lib/Drupal/Core/StackMiddleware/KernelPreHandle.php(48): Drupal\Core\StackMiddleware\Session->handle(Object(Symfony\Component\HttpFoundation\Request), 1, true)
#19 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/modules/page_cache/src/StackMiddleware/PageCache.php(106): Drupal\Core\StackMiddleware\KernelPreHandle->handle(Object(Symfony\Component\HttpFoundation\Request), 1, true)
#20 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/modules/page_cache/src/StackMiddleware/PageCache.php(85): Drupal\page_cache\StackMiddleware\PageCache->pass(Object(Symfony\Component\HttpFoundation\Request), 1, true)
#21 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/modules/ban/src/BanMiddleware.php(50): Drupal\page_cache\StackMiddleware\PageCache->handle(Object(Symfony\Component\HttpFoundation\Request), 1, true)
#22 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/lib/Drupal/Core/StackMiddleware/ReverseProxyMiddleware.php(48): Drupal\ban\BanMiddleware->handle(Object(Symfony\Component\HttpFoundation\Request), 1, true)
#23 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/lib/Drupal/Core/StackMiddleware/NegotiationMiddleware.php(51): Drupal\Core\StackMiddleware\ReverseProxyMiddleware->handle(Object(Symfony\Component\HttpFoundation\Request), 1, true)
#24 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/vendor/stack/builder/src/Stack/StackedHttpKernel.php(23): Drupal\Core\StackMiddleware\NegotiationMiddleware->handle(Object(Symfony\Component\HttpFoundation\Request), 1, true)
#25 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/core/lib/Drupal/Core/DrupalKernel.php(708): Stack\StackedHttpKernel->handle(Object(Symfony\Component\HttpFoundation\Request), 1, true)
#26 /home/bavramor/data/Webdesign/Webseiten/landwehrcie-backup-drupal9/web/index.php(19): Drupal\Core\DrupalKernel->handle(Object(Symfony\Component\HttpFoundation\Request))
#27 {main}
?>- Anmelden oder Registrieren um Kommentare zu schreiben
Die Fehlermeldung heißt für
am 02.05.2022 - 22:36 Uhr
Die Fehlermeldung heißt für mich, daß das Object $node->body nicht existiert und deshalb dazu auch kein Wert format zugefügt werden kann. Das müßte jetzt genauer gedebuggt werden, um den Grund zu finden. Vermutlich ein spezieller Node oder Node-Typ. Also solltest Du vor der Zuweisung überprüfen, ob $node->body existiert.
.
Werner
drupal-training.de
Moderator und Drupal Trainer
* - - - - - - - - - - - - - - - - - - - - - - - - - - - *
Also die Fehlermeldung sagt
am 22.07.2022 - 13:44 Uhr
Also die Fehlermeldung sagt es ja,
$node->bodygibt es anscheinend nicht. Gibt es das body Field denn generell noch bei dem Inhaltstypen? Vielleicht baust Du eine Prüfung ein:if (isset($node->body)) {$node->body->format = 'full_html';
}
Aber mal eine Frage: der Funktion wird ein Parameter $node übergeben, dieser wird aber gleich zu Beginn mit der Zeile
$node = $node_storage->load($nid);wieder überschrieben, $node ist dann wohl ein Mustervertrag oder so was. Und wenn
if ($operation == 'edit')ist, dann wird $node noch einmal überschrieben, und zwar mit einem wieder neu geladenen Node, dessen Id aus der URL genommen wird. Mein Tipp: im späteren Verlauf der Funktion wird mit $node irgendetwas anderes erwartet, als dann da ist. Der Inhalt einer Variablen sollte innerhalb einer Methode so nicht überschrieben werden, weil man in der Mitte der Funktion eigentlich nicht mehr weiß, was $node eigentlich ist.