Refresh does not work on aclrequest list

[CrosRoad95]

Describe the bug
/refresh command does not refresh aclrequest list in meta.xml

To reproduce

1. create resource with meta.xml content eg.

<aclrequest>
<right name="function.startResource" access="true" />
<right name="function.stopResource" access="true" />
<right name="function.setElementPosition" access="true" />
</aclrequest>

2. type /refresh and aclrequest list X where X is your resource name
3. modify aclrequest list, eg remove one right
4. type /refresh and aclrequest list X again - nothing changed
5. rename resource X to Y
6. type /refresh and aclrequest list Y again - you will see you changes

Expected behaviour

Screenshots
/

Version
1.5.8 release 20698

Additional context
/

[PerikiyoXD]

refreshall
[07:59:07] Resource 'resources1824' changed, reloading
[07:59:07] Resource 'resources1824' requests some acl rights. Use the command 'aclrequest list resources1824'
[07:59:07] New resource 'resources1824' loaded
[07:59:07] Resources: 216 loaded, 0 failed
aclrequest list resources1824
[07:59:13] aclrequest: resources1824 [pending] for function.startResource
[07:59:13] aclrequest: resources1824 [pending] for function.stopResource
refreshall resources1824
[07:59:26] Resource 'resources1824' changed, reloading
[07:59:26] Resource 'resources1824' requests some acl rights. Use the command 'aclrequest list resources1824'
[07:59:26] New resource 'resources1824' loaded
[07:59:26] Resources: 216 loaded, 0 failed
aclrequest list resources1824
[07:59:36] aclrequest: resources1824 [pending] for function.startResource
[07:59:36] aclrequest: resources1824 [pending] for function.stopResource
[07:59:36] aclrequest: resources1824 [pending] for function.setElementPosition

// Load the complete list of resources and create their objects
// DOES NOT reload already loaded resources, we need a special function for lua for that (e.g. reloadResource)
// Talidan: yes it did, noob.  Not as of r897 - use bRefreshAll to do already loaded resources.
bool CResourceManager::Refresh(bool bRefreshAll, const SString strJustThisResource, bool bShowTiming)

Not as of r897 - use bRefreshAll to do already loaded resources.

So "refreshall" triggers resource loading, so this is nonsensical?

[PerikiyoXD]

Actual solution is to use /refreshall as:
refresh command docs
refreshall command docs

So this doesn't apply. I'd evaluate if extra code introduced by d0e210a shall be reverted.