search

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

Orchard.Taxonomies - Merge feature implemented but not surfaced #6399

Open rtpHarry opened 8 years ago

rtpHarry commented 8 years ago

Dev Branch

I checked the issues for any history on it but I can't see it being abandoned so it seems like an oversight.

Merge feature exists but isn't surfaced

Merge feature exists in code and has views etc but isn't listed in the bulk operations drop down.

Typos in authorize check

Additionally, the Authorize check will return wrong message when it fails, and checks the wrong permission:

if (!Services.Authorizer.Authorize(Permissions.ManageTerms, T("Couldn't delete term")))

Should say:

if (!Services.Authorizer.Authorize(Permissions.MergeTerms, T("Couldn't merge terms")))
sebastienros commented 8 years ago

If the code works then please do.