Empty fields should be nulls

Open on Drupal.org →

Created on 11 August 2024, 5 months ago

Problem/Motivation

If you try to index an entity that has a null value in a string field that is setup as an Azure string data type, you get:
The request is invalid. Details: Cannot convert the literal 'False' to the expected type 'Edm.String'.

Proposed resolution

The problem is the use of php's end function in the document param builder. This function returns FALSE if called on an empty array, but we should be passing on a NULL to Azure in this case no a FALSE

Remaining tasks

None.

User interface changes

None.

API changes

None.

Data model changes

None.

🐛 Bug report

Status

Needs review

Version

1.0

Component

Code

Created by

🇬🇧United Kingdom jonathanshaw Stroud, UK

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!3Empty fields should be nulls
Open
🇬🇧United Kingdom jonathanshaw
updated 5 months ago

Comments & Activities

Issue created by @jonathanshaw
Merge request !3#3467433: Empty fields should be null → (Open) created by jonathanshaw
Pipeline finished with Success
5 months ago
Total: 135s
#250728
Comment 5 months ago →
🇬🇧United Kingdom jonathanshaw Stroud, UK
Status changed to Needs review 5 months ago3:31pm 11 August 2024
Comment 5 months ago →
🇬🇧United Kingdom jonathanshaw Stroud, UK
Comment 4 months ago →
🇧🇪Belgium vodde83
interx → credited vodde83 → .
Comment 4 months ago →
🇧🇪Belgium interx Flanders
Thanks you for the contribution.

This should be fixed in this commit from @vodde83.
Comment 3 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024