search

dpi / rng

RNG is a Drupal module enabling people to register for events.
https://www.drupal.org/project/rng
GNU General Public License v2.0
15 stars 23 forks source link

Postgres tests failing due to string usage in integer field. #50

Open dpi opened 9 years ago

dpi commented 9 years ago

DER issue: Support non-numeric entity ID's

Test: https://www.drupal.org/pift-ci-job/31805

exception: [Uncaught exception] Line 658 of core/lib/Drupal/Core/Database/Connection.php: Drupal\Core\Database\DatabaseExceptionWrapper: SQLSTATE[22P02]: Invalid text representation: 7 ERROR: invalid input syntax for integer: "registration_type_a" LINE 5: ...d (courier_template_collection.owner__target_id = 'registrat...

Stack:

... courier_entity_predelete(Object, 'registration_type') call_user_func_array('courier_entity_predelete', Array) Drupal\Core\Extension\ModuleHandler->invokeAll('entity_predelete', Array) Drupal\Core\Config\Entity\ConfigEntityStorage->invokeHook('predelete', Object) Drupal\Core\Entity\EntityStorageBase->delete(Array) Drupal\Core\Entity\Entity->delete() Drupal\rng\Tests\RegistrationTypeTest->testRegistrationTypes() ...

dpi commented 8 years ago

Can sort-of workaround this by checking if deleted entities are not bundle entities. This still means that entities with non numeric ID's are still unsupported.