Add possibility to set "required" RBAC conditions

[Convly]

What does it do?

Add a new willRegister hook for the permission engine. This hook exposes a context which allow to add raw conditions to the permission.

A handler for this hook would then be able to add condition with or / and logic to the final permission
Example:

const handler = context => {
  const { permission, condition, user } = context;

  // ...

  condition
    .and({
      foo: {
        $in: ['bar', 'foobar'],
      }
    })
    .or({ bar: 'foo' });
}

All conditions added with an and are required to pass for the condition to be valid, whereas only one of the conditions added with an or need to pass.

Why is it needed?

Currently, all conditions are evaluated with an or logic. It can lead to issues when multiples conditions need to be matched (eg: is-creator (rbac) + has-locale-access (i18n) => if one of the condition is matched, then the other doesn't matter)

Related issue(s)/PR(s)

fix #10159

Notes

I'll submit a PR after this one to add more complete unit tests for the permission engine.

[codecov]

Codecov Report

Merging #10185 (7f941cd) into master (3198d19) will decrease coverage by 0.03%.
The diff coverage is 64.38%.

@@            Coverage Diff             @@
##           master   #10185      +/-   ##
==========================================
- Coverage   57.86%   57.82%   -0.04%     
==========================================
  Files         183      184       +1     
  Lines        6348     6393      +45     
  Branches     1392     1395       +3     
==========================================
+ Hits         3673     3697      +24     
- Misses       2215     2233      +18     
- Partials      460      463       +3     

Flag	Coverage Δ
front	∅ <ø> (∅)
unit	57.82% <64.38%> (-0.04%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
packages/strapi-admin/domain/condition/index.js	100.00% <ø> (ø)
...s/strapi-admin/services/permission/engine-hooks.js	24.13% <24.13%> (ø)
packages/strapi-admin/services/role.js	94.70% <33.33%> (-1.25%)	⬇️
...n/services/permission/permissions-manager/index.js	95.00% <87.50%> (-2.06%)	⬇️
...ackages/strapi-admin/services/permission/engine.js	96.29% <96.55%> (+5.75%)	⬆️
...es/permission/permissions-manager/query-builers.js	100.00% <100.00%> (ø)
packages/strapi-utils/lib/build-query.js	16.45% <100.00%> (ø)
...ages/strapi-utils/lib/convert-rest-query-params.js	93.82% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3198d19...7f941cd. Read the comment docs.

[alexandrebodin]

LGTM