mysql connector create method overriding default values about loopback-connector-mysql HOT 5 CLOSED

We need to differentiate between undefined and null. If a property is set to null, we should pass NULL to MySQL. If a property is not present or undefined, we should skip the column. Can you create a patch?

from loopback-connector-mysql.

Wouldn't the above fix remove also the values that have been explicitly set to null?

from loopback-connector-mysql.

Wouldn't the above fix remove also the values that have been explicitly set to null?

@rnybergh That's what I'm trying to avoid.

from loopback-connector-mysql.

Wouldn't this do it?

if('NULL'!== value && data[key]!==undefined) {
   fields.push(self.columnEscaped(model, key) + ' = ' + value);
}

from loopback-connector-mysql.

This issue should be fixed on the latest version of the connector. The user can specify a default value of a property on the model definition, and the create method will honour the default value if the property is not specified upon the create request.

Here is a sample code snippet:

model-definition.json

{
  "name": "Employee",
  "base": "PersistedModel",
  "idInjection": false,
  "options": {
    "validateUpsert": true
  },
  "properties": {
    "id": {
      "type": "number",
      "required": true,
      "id": true
    },
    "role": {
      "type": "string",
      "default": "Software Developer"
    },
    "name": {
      "type": "string",
      "required": true
    }
  },
  "validations": [],
  "relations": {},
  "acls": [],
  "methods": {}
}

boot-script.js

'use strict';
var util = require('util');

module.exports = function(app) {
  var db = app.dataSources.mysqlDs;
  var Employee = app.models.Employee;

  db.automigrate(function(err) {
    if (err) throw err;
    Employee.create({id: 1, name: 'Bob'}, function(err, result) {
      if (err) throw err;
      console.log('\nCreated instance: ' + util.inspect(result, 4));
    });
  });
};

console output

sakibs-mac:mysql-51 ssh$ node .
Web server listening at: http://0.0.0.0:3000
Browse your REST API at http://0.0.0.0:3000/explorer

Created instance: { id: 1, name: 'Bob', role: 'Software Developer' }

database output

mysql> select * from Employee;
+----+--------------------+------+
| id | role               | name |
+----+--------------------+------+
|  1 | Software Developer | Bob  |
+----+--------------------+------+

Closing this issue as a result. Feel free to reopen if needed.

from loopback-connector-mysql.