More checks on insert/update about spine-database-api HOT 9 OPEN

In GitLab by @manuelma on Dec 14, 2018, 21:14

changed title from More checks on inser/update to More checks on inser{+t+}/update

from spine-database-api.

In GitLab by @PekkaSavolainen on Dec 17, 2018, 11:17

changed the description

from spine-database-api.

In GitLab by @PekkaSavolainen on Dec 17, 2018, 11:26

I added one task (Check that input is valid JSON). I'm not sure if this is implemented already but it just means that it should not be possible to give input that is invalid JSON, like [2, 3, 4 or something. This could just be a try-except check. Something like the following.

try:
    json.load(someinput)
except json.JSONDecodeError:
    # not valid JSON
    return False
return True

There seems to be some checks for this in the Toolbox already, so this might be done already.

from spine-database-api.

In GitLab by @manuelma on Jan 24, 2019, 16:06

changed the description

from spine-database-api.

In GitLab by @manuelma on Jan 24, 2019, 16:06

changed the description

from spine-database-api.

In GitLab by @manuelma on Feb 28, 2019, 16:15

I wonder what's the best way to implement the forbidden names? At the moment, if one has a parameter called, e.g., 'length', then SpineModel.JuMP_all_out fails with the following error message:

ERROR: LoadError: error in method definition: function Base.length must be explicitly imported to be
 extended

My guess is it would be difficult to keep a blacklist in spinedatabase_api and perhaps it's better to raise this kind of errors in julia? We could rework the error message to instruct the user to change their parameter name....

from spine-database-api.

In GitLab by @jkiviluo on Mar 1, 2019, 12:15

The blacklist should be tool/language specific. It would be nicer to catch it on the Data Store side already, but it's probably impractical. Therefore, the forbidden names could also be in JuMP_all_out for all Julia based tools. It might be that this problem is actually Julia specific since we turn our objects and parameters into function calls. That would not be the case with all languages.

from spine-database-api.

In GitLab by @manuelma on Apr 9, 2019, 00:55

mentioned in commit 178d747

from spine-database-api.

In GitLab by @manuelma on May 30, 2019, 19:12

marked the task Check that input is valid JSON as completed

from spine-database-api.