media_oembed_permission() creates the 'administer media oembed' permission, but this isn't used anywhere. The 'access arguments' for the settings form uses the permission 'administer site configuration'. Patch attached changes it to 'administer media oembed' since I think that's what was intended by creating that custom permission.

CommentFileSizeAuthor
#2 use-correct-permission-2619236-1.patch495 bytesshaundychko

Comments

ShaunDychko created an issue. See original summary.

shaundychko’s picture

Status: Active » Needs review
StatusFileSize
new495 bytes
devin carlson’s picture

Version: 7.x-2.4 » 7.x-2.x-dev
Status: Needs review » Fixed

Thanks for the patch!

The permission is currently required to administer providers but I agree that it should also be used for general settings.

Tested #2 and committed to Media: oEmbed 7.x-2.x.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.