OrchardCMS / Orchard

Orchard is a free, open source, community-focused Content Management System built on the ASP.NET MVC platform.
https://orchardproject.net
BSD 3-Clause "New" or "Revised" License
2.37k stars 1.12k forks source link

Added orchardFeature attribute where it was missing in Orchard.Roles #8680

Closed MatteoPiovanelli-Laser closed 1 year ago

MatteoPiovanelli-Laser commented 1 year ago

We found that a few files we touched (https://github.com/OrchardCMS/Orchard/commit/97288315564d6aed63c6a1925628669b030ab094) had an issue: they did not have the OrchardFeature attribute, so the ahndler they implemented could be invoke out of turn. For example, it would be triggered when the "admin" user is created during the "default" startup recipe that creates a tenant, logging an error as the workflow feature is not active at that point.

This PR fixes that.