AssertionError: assert(is_string($field)) in assert()

The website encountered an unexpected error. Try again later. AssertionError: assert(is_string($field)) in assert() (line 330 of core/lib/Drupal/Core/Database/Query/Merge.php). Drupal\Core\Database\Query\Merge->key(Array) (Line: 450) Drupal\abjs\Form\AbjsTestForm->saveTest(Array, Object) call_user_func_array(Array, Array) (Line: 105) Drupal\Core\Form\FormSubmitter->executeSubmitHandlers(Array, Object) (Line: 43) Drupal\Core\Form\FormSubmitter->doSubmitForm(Array, Object) (Line: 589) Drupal\Core\Form\FormBuilder->processForm('abjs_test', Array, Object) (Line: 321) Drupal\Core\Form\FormBuilder->buildForm(Object, Object) (Line: 73) Drupal\Core\Controller\FormController->getContentResult(Object, Object) call_user_func_array(Array, Array) (Line: 123) Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->Drupal\Core\EventSubscriber\{closure}() (Line: 593) Drupal\Core\Render\Renderer->executeInRenderContext(Object, Object) (Line: 121) Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->wrapControllerExecutionInRenderContext(Array, Array) (Line: 97) Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->Drupal\Core\EventSubscriber\{closure}() (Line: 183) Symfony\Component\HttpKernel\HttpKernel->handleRaw(Object, 1) (Line: 76) Symfony\Component\HttpKernel\HttpKernel->handle(Object, 1, 1) (Line: 53) Drupal\Core\StackMiddleware\Session->handle(Object, 1, 1) (Line: 48) Drupal\Core\StackMiddleware\KernelPreHandle->handle(Object, 1, 1) (Line: 28) Drupal\Core\StackMiddleware\ContentLength->handle(Object, 1, 1) (Line: 32) Drupal\big_pipe\StackMiddleware\ContentLength->handle(Object, 1, 1) (Line: 116) Drupal\page_cache\StackMiddleware\PageCache->pass(Object, 1, 1) (Line: 90) Drupal\page_cache\StackMiddleware\PageCache->handle(Object, 1, 1) (Line: 48) Drupal\Core\StackMiddleware\ReverseProxyMiddleware->handle(Object, 1, 1) (Line: 51) Drupal\Core\StackMiddleware\NegotiationMiddleware->handle(Object, 1, 1) (Line: 36) Drupal\Core\StackMiddleware\AjaxPageState->handle(Object, 1, 1) (Line: 51) Drupal\Core\StackMiddleware\StackedHttpKernel->handle(Object, 1, 1) (Line: 709) Drupal\Core\DrupalKernel->handle(Object) (Line: 19)

Comments & Activities

Issue created by @alberto56

Comment 23 days ago →

🇨🇦Canada alberto56

It seems that line 330 of core/lib/Drupal/Core/Database/Query/Merge.php has this code:


  /**
   * Sets a single key field to be used as condition for this query.
   *
   * Same as \Drupal\Core\Database\Query\Merge::keys() but offering a signature
   * that is more natural for the case of a single key.        
   *
   * @param string $field
   *   The name of the field to set.                     
   * @param mixed $value
   *   The value to set into the database.                            
   *
   * @return $this
   *                                                        
   * @see \Drupal\Core\Database\Query\Merge::keys()
   */                                                    
  public function key($field, $value = NULL) {
    assert(is_string($field));          
    $this->keys([$field => $value]);
    return $this;                                                          
  }

But line 450 of src/Form/AbjsTestForm.php has the following code:

        $this->database->merge('abjs_test_condition')
          ->key(['tid' => $tid, 'cid' => $cid])
          ->fields(['tid' => $tid, 'cid' => $cid])->execute();

In other words, AbjsTestForm. is passing an array [tid=>tid, cid=>cid] to a function that fails if you do not pass a string to it.

Comment 23 days ago →
🇨🇦Canada alberto56
This assertion was added to Drupal core in 📌 [11.x] Adjust Database API to remove deprecated Drupal 10 code in Drupal 11 Fixed for Drupal 11.

So presumably this module works in Drupal 10, although I have not tested it.

The problematic code exists in abjs commit 6aaf7c4 - Drupal 11 compatiblility fixes, which to my understanding is the latest commit.
Comment 23 days ago →
🇨🇦Canada alberto56
Confirming the module works on Drupal 10, but not Drupal 11. Changing the title accordingly

AssertionError: assert(is_string($field)) in assert()

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments & Activities