[drupal-devel] [feature] Add configuration option to show blocks only on pages of certain node type

[andremolnar]

 Project:      Drupal
 Version:      cvs
 Component:    block.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  andremolnar
 Reported by:  andremolnar
 Updated by:   andremolnar
 Status:       patch
 Attachment:   http://drupal.org/files/issues/block_module_2.patch (7.61 KB)

Okay - here is an updated patch.

1) Cleaned up the coding style for if else statements (my code
'beautifier' insisted that the other style was 'more beauthiful') ;-)
2) Changed the 'type' column to 'types'
3) Changed the code to reflect new column name
4) Changed variable names (e.g. $typematch -> $type_match)
5) Changed radios to checkboxes
6) Changed type matching block to handle multiple choices from
checkboxes

Quick question regarding coding style for concatinated strings (didn't
see this in the documentation) - do we:

'string string string'. $foo .'string'  OR
'string string string' . $foo . 'string'


I tend to do both - but lean towards the latter.

andre

p.s. blocks based on roles would be another patch if I have time ;-)

andremolnar



Previous comments:
------------------------------------------------------------------------

January 24, 2005 - 01:47 : andremolnar

Attachment: http://drupal.org/files/issues/block_module_1.patch (8.14 KB)

This patch would allow adminstrators to configure blocks to show up only
pages of a specified node type.

This patch is being submitted in response to feedback on the proposed
changes to the book module http://drupal.org/node/14120.  Feedback
indicated that if the block generation associated with books is
changed, administrators will still require the ability to specify that
newly created book blocks behave as the existing book blocks do (i.e.
only show up on book pages).

This patch goes one step further and would allow an admin to show a
book block (or any block) on any type of page.

While independent of 14120 - this patch in conjuntion with 14120 would
allow the removal of a large piece of block generation code from the
book module.

Comments and feedback, as always, are welcome.

andre

------------------------------------------------------------------------

January 24, 2005 - 01:48 : andremolnar

Attachment: http://drupal.org/files/issues/blocks_db_change.patch (1.98 KB)

Associated db changes required for this patch.

andre

------------------------------------------------------------------------

January 24, 2005 - 08:17 : FactoryJoe at civicspacelabs.org

I hope at some point we can go beyond just "showing blocks on certain
node types" to actually determine what blocks to show at the "per page"
level (although a page in and of itself is somewhat nebulous).

Essentially, if I have '/home/about/' I would like to be able to target
that one page a "add blocks" to it in various regions. Of course, doing
this in a cascading way would also be great: if I have '/home/about/'
to which I add Blocks X, Y and Z and then I have '/home/about/bios/', I
could disable X and Y but leave Z alone and it would be "inherited".

I realize that there was a huge discussion about this topic that I
didn't completely follow, so hopefully my comments aren't too far off
base.

------------------------------------------------------------------------

January 24, 2005 - 10:17 : Steven

Chris, have you seen the block configuration in HEAD? Drumm developed
that patch...

------------------------------------------------------------------------

January 24, 2005 - 15:12 : Dries

The coding style (placement of spaces and brackets) needs a bit of work,
but the functionality is a welcome addition that I would commit to
HEAD.

Plus, I suggest renaming some variables for readability.  The following
can be improved:


1. $typematch -> $type_match
2. $matched -> $page_match

For consistency with the 'page matching', I suggest that you transform
the radio-buttons to checkboxes so multiple node types can be selected
and that we rename 'block.type' to 'block.types' (cfr. 'block.pages',
plural).  On drupal.org, I'd like to disable some block on 'project
issue' pages which would not be possible with your patch, but which
would be possible if checkboxes were used. 

Lastly, avoid the word 'node' in output: use 'post' instead.

On a related note, there is also an interest in being able to display
blocks on a role-base so if time permits, you might want to introduce a
'block.roles' column too. ;-)

I vaguely remember a patch by Neil to tidy up the block table: not sure
if that is still around and whether that needs to be considered.

-- 
View: http://drupal.org/node/16074
Edit: http://drupal.org/project/comments/add/16074