Stores should not have to implement the Add() method

Currently, I am creating a store for plugins (!230 (merged)) within the controller. Looking at our other store implementation, e.g., PNDs, Devices, etc., one can see that each is based on a generic store. But since some of the stores need persistence and others don't, there is some redundant code for the Add() method.

Description

Expected Behavior

Actual Behavior

Possible Fix

It would make more sense to add a persist() method within the generic store. By default, this method would do nothing, but the implementation could be done there if a store needs persistence.

Steps to Reproduce

Context

Your Environment

  • Version used:
  • Environment name and version (e.g. go v1.16.3 on FreeBSD 13.0-current):
  • Server type and version:
  • Operating System and version:
Edited by Ghost User