Drivers - add() needs id parameter

[benjaminplanche]

The add function of the drivers also needs to get as parameter the id field of the sensor, which is its ID in the DB.

The driver should pass this information to the sensor (or its base), so it can send valid requests to push new measures.

(Indeed, customId can't be used since we can't be sure it is unique - unless we also pass the type field).

[dbuldon]

In a general case, I agree with you, and I can add the "id" parameter to the function.
However, in the case of the sunspots, that method is only in charge of telling the base station that one particular sensor is allowed to send data.

For the base station, the id of a sensor (primary key of the table Sensor), has no meaning, but the mac address (first part of the custom id), identifies completely one sensor, so using it is the most logical way of controlling the sensors´ permissions in the base station side.
Also in the case of sunspots, customId is unique, as it contains the mac address of the device.

However, I have no problem in include id field in that method, if in this way we can make it more general