DB schema for cmrf_core_call incorrect for request and reply columns

CiviMRF / CMRF_Abstract_Core

Abstract implementation for the the CMRF Core. Should be included into the concrete module implementation

GNU Affero General Public License v3.0

1 stars 4 forks source link

Closed jensschuppe closed 5 years ago

jensschuppe commented 5 years ago

In 07d80b4d7b71c1476b618bab9f94b4e4ac15eb87, NOT NULL was set to TRUE, which is

inconsistent with the latest hook_update_N() in the Drupal module (See commit 4862cd1f403f9acc44e0eff1c7ba84968891978c)
conceptually wrong and leads to default value exceptions on new installations

Proposed fix is to set NOT NULL for request and reply columns to FALSE.

jensschuppe commented 5 years ago

Actually, request seems to not have to be NOT NULL - but should be in sync with the Drupal module's latest update hook.