Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add new cache capabilities method(s) #578

Open
following5 opened this issue Jan 17, 2019 · 2 comments
Open

Add new cache capabilities method(s) #578

following5 opened this issue Jan 17, 2019 · 2 comments

Comments

@following5
Copy link
Contributor

When designing services/caches/capabilities, I missed that capabilities can be withdrawn. E.g. OCPL already allows to disable cache sizes. A disabled size can no longer be assigned to caches, but can still be present with existing caches.

So there can be different capabilities for submitting cache data than for searching. This applies to:

The is NO problem with:

  • cache statuses - search-only capability, cannot be directly edited
  • has_ratings - search-only capability, cannot be directly edited
  • password_max_length - edit-only capability, not seachreable

This may be solved by splitting between

  • services/caches/capabilities (submitting)
  • services/caches/search/capabilities (searching)

Search capabilities would be the union of all existing and historic submit capabilties. Additionally, we could supply information about existing cache properties, so that developers can optimize user interfaces by hiding unused options.

(Note that there is no issue with services/logs/capabilities, which are defined for submitting/editing logs only. Also there is no issue with editing caches or logs that have redrawn properties, e.g a no longer allowed cache sizes - both methods allow to retain them.)

This issue blocks #570.

@following5
Copy link
Contributor Author

following5 commented Jan 17, 2019

This is critical because the interface is flawed, but not in use yet. Should fix before anyone uses it.

@following5
Copy link
Contributor Author

Removed services/caches/capabilities by 72e03f7.

@following5 following5 changed the title Cache capabilities can differ for submitting and searching Add new cache capabilities method(s) Jan 17, 2019
@following5 following5 removed their assignment Nov 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant