This patch is from HEAD and adds the "administer javascript aggregator" permission.

CommentFileSizeAuthor
#2 327717_js_aggregator_administer.patch2.02 KBchristefano
js_aggregator_administer.patch3.07 KBrobloach

Comments

robloach’s picture

robloach
he/him
commented
Status: Needs review » Fixed

Committed.

christefano’s picture

christefano commented
Status: Fixed » Needs review
StatusFileSize
new 327717_js_aggregator_administer.patch2.02 KB

Is a new permission really necessary? It just adds more work.

This patch makes JavaScript Aggregator use the 'administer site configuration' permission, which is already necessary in order to see the Performance settings.

robloach’s picture

robloach
he/him
commented
Title: Administer JavaScript Aggregator permission » Administer Site Configuration permission
Version: 5.x-0.x-dev » 5.x-1.x-dev
Status: Needs review » Fixed

Sounds reasonable to me! Committed: http://drupal.org/cvs?commit=150474 .... Thanks!

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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