Core block module's block_admin_configure() provides an invocation for hook_block_configure() (see code). This allows other modules to easily modify and extend the block configuration form.

This should be implemented in mongodb_block_ui.admin.inc's mongodb_block_ui_admin_configure() function so that existing modules can easily integrate with mongodb blocks.

CommentFileSizeAuthor
#1 hook_block_configure-2087117-1.patch828 bytesjyee
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jyee’s picture

jyee CreditAttribution: jyee commented
Status: Active » Needs review
FileSize
hook_block_configure-2087117-1.patch828 bytes

patch attached.

marcingy’s picture

marcingy CreditAttribution: marcingy commented
Category: bug » task
Status: Needs review » Reviewed & tested by the community

Looks good and replicates exactly what core does. Have changed classification as this is effectively a new nice too have rather than a bug as such.

  • fgm committed 4dd22e3 on 7.x-1.x 
    Issue #2087117 by jyee: Added hook_block_configure in block configure...
fgm’s picture

fgm
Primary language French
Location Paris, France
CreditAttribution: fgm commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Replicates core behaviour, does not increase the already high number of test fails, no one objected to RTBC in 2 years: committed to 7.x-1.x. Thanks.

This being said, it would probably be more useful to address test failures before changing features like this.

Status: Fixed » Closed (fixed)

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