dm4-webclient: number types are persisted as string values..

[Malte]

Despite having set a "dm4.core.number" Data Type the webclient sends those topics values back to the REST API as numeric values wrapped in a string, consequently making neo4j initialize such values as of type String.

Making dm4-reviews compatible with 4.4 has brought up this issue again, see addOne() and substractOne() at the bottom of ​https://github.com/mukil/dm4-reviews/blob/master/src/main/java/org/deepamehta/plugins/review/ReviewPlugin.java

Hope this helps!

[jri]

Confirmed!

I broke serialization/storage of numbers in DM 4.2 (almost a year ago!) in this commit:
​https://github.com/jri/deepamehta/commit/90546faecf258ff65b8a7f9cc342d71d1e302b36

In DM 4.1.3 it still worked correctly.

Thank you very much for reporting!

[Malte]

BTW: It would be of great help if a patch for this would make up for a 4.4.2 and not force us to update to 4.5 to make use of numbers in the webclient.

Otherwise, it would be great if you can some infos here on how to fix/patch this issue in the dm4-webclient source code, since it is of matter in a new project i am currently working on.

Thanks & Cheers!

[jri]

OK, I'll fix that nasty bug and release 4.4.2 very soon.
Thank you for insisting :-)

[Jörg Richter <jri@…>]

In c4185114886423eb2b91602a14ff090f4f201c7c/deepamehta:

Webclient: fix number fields (#750).

The Webclient reads out input in "Number" fields properly as numbers.
As a consequence numbers are properly serialized (as JSON numbers) and stored/retrieved as numbers.
(Number handling was broken in DM 4.2 in the course of supporting aggregated Number child types.)

Note: numbers are also properly handled when choosen/entered from/in a combobox (when defined via aggregation).

Thank you, Malte, for reporting!

You can test it in the "number-fix" branch.
If it works for you I can release it as 4.4.2

Webclient API

1 new method in TopicType and AssociationType:

function is_number()

Returns true if the type's data type is "Number".

1 changed method:

dm4c.render.form_element(...)

It can also be used to render text areas.

1 changed method:

dm4c.render.form_element_function(form_element, page_model)

It takes a page_model as the 2nd parameter.

See #750.

[Jörg Richter <jri@…>]

In 52891f5340bc90a790c350d935d63f731a4cda76/deepamehta:

Webclient: fix comboboxes (#750).

See #750.

[Malte]

Just tested it and editing numbers via webclient are functional again! Thanks a lot!