You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In order to eventually sort our our circular dependency issues and get the DKAN modules organized in a more intuitive way (not to mention making the code more comprehensible) we need to collapse these two very redundant classes/objects into a single thing. The DatastoreResource class has very little value; it does not reference any other classes or services from the Datastore module and just adds a few more properties that are just built from the DataResource properties.
This will take a bit of trial and error to get right, but will consist of, generally
Figuring out how to reproduce any additional fuctionality from DatastoreResource in the DataResource class. This may involve some util-type functions that stay in Datastore and process info retrievevd from DataResource`.
Changing all uses of DatastoreResource throughout the Datastore module to use DataResource instead.
Deprecating DatastoreResource but keeping for now as a shim to prevent any custom or contrib code using it from breaking.
The text was updated successfully, but these errors were encountered:
In order to eventually sort our our circular dependency issues and get the DKAN modules organized in a more intuitive way (not to mention making the code more comprehensible) we need to collapse these two very redundant classes/objects into a single thing. The
DatastoreResource
class has very little value; it does not reference any other classes or services from the Datastore module and just adds a few more properties that are just built from the DataResource properties.This will take a bit of trial and error to get right, but will consist of, generally
DatastoreResource
in theDataResource
class. This may involve some util-type functions that stay in Datastore and process info retrievevd from DataResource`.DataResource
instead.The text was updated successfully, but these errors were encountered: