Page MenuHomeSealhub

'when' access strategy - Ref T805
ClosedPublic

Authored by kuba-orlik on Mar 30 2018, 16:13.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 27, 04:17
Unknown Object (File)
Wed, Mar 27, 04:17
Unknown Object (File)
Wed, Mar 27, 04:17
Unknown Object (File)
Wed, Mar 27, 02:06
Unknown Object (File)
Mon, Mar 25, 09:16
Unknown Object (File)
Wed, Mar 20, 13:29
Unknown Object (File)
Mon, Mar 11, 22:17
Unknown Object (File)
Fri, Mar 8, 20:20

Details

Reviewers
piotr-ptaszynski
Group Reviewers
Unknown Object (Project)
Maniphest Tasks
Unknown Object (Maniphest Task)
Commits
rSbadbf12ea3ba: 'when' access strategy - Ref T805
rSadb9f6ba1fd7: 'when' access strategy - Ref T805
Summary

See tests for usage

Diff Detail

Repository
rS Sealious
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kuba-orlik created this revision.
kuba-orlik added a task: Unknown Object (Maniphest Task).Mar 30 2018, 16:31
kuba-orlik changed the visibility from "All Users" to "Public (No Login Required)".
piotr-ptaszynski accepted this revision as: Unknown Object (Project).Mar 30 2018, 19:18
piotr-ptaszynski subscribed.

Bardzo fajna metastrategia. Są pewne uwagi, ale raczej non-blocking.

lib/app/base-chips/access-strategy-types/or.js
23

O kurde nie przyszło mi to do głowy +1 !

lib/app/base-chips/access-strategy-types/when.js
50

(question)
Ok, wypisujemy błąd - ale równocześnie go połykamy, tak ma być?

lib/app/base-chips/special_filters/IsReferencedByResourcesMatching.js
36

(could)
Mam wątpliwości czy to jest dobra nazwa dla filtrów - jako twórca nazwy getRestrictionQuery kierowałem się tym, że "restriction" kojarzy się z ograniczaniem dostępu, a to nie jest zadanie filtru. Na przykład getFilteringQuery pasowałoby bardziej.

lib/datastore/query.js
147

(should)
W klasie Query są już metody statyczne zadeklarowane za pomocą static i trzymałbym się tego.

bartosz-gordon added inline comments.
lib/app/base-chips/access-strategy-types/or.js
23

Lubię widzieć takie zmiany :D

kuba-orlik marked 5 inline comments as done.

Zastosowałem się do sugestii, wprowadziłem też kilka zmian - upubliczniłem interfejs SpecialFilters, nie trzeba go teraz ręcznie require'ować

lib/app/base-chips/access-strategy-types/when.js
50

Uuu tego tu powinno nie być, to artefakt po debuggowaniu.

kuba-orlik marked an inline comment as done.
  • Review fixes, making the filters interface public

Oddaję do ponownego review ;)

This revision is now accepted and ready to land.Apr 4 2018, 09:15
This revision was automatically updated to reflect the committed changes.