Closed (fixed)
Project:
Drupal core
Component:
block.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
2 Jul 2005 at 04:54 UTC
Updated:
12 Oct 2005 at 22:20 UTC
Jump to comment: Most recent file
- I create a new block with some HTML in it.
- I configure the block, setting Visibility to "Show on only the listed pages." and list just "<front>".
- I save and get the following warning:
warning: preg_match() [function.preg-match]: Unknown modifier '3' in /home/www/site/modules/block.module on line 474.
The block does not show up on the front page.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | block_3.patch | 850 bytes | Thiago Bastos |
Comments
Comment #1
Thiago Bastos commentedHere's a patch. preg_quote(variable_get('site_frontpage', 'node'), '/') did the trick:
Comment #2
dries commentedCan you upload your patch?
Comment #3
Thiago Bastos commentedSorry, here's the attachment.
I wasn't fully aware of the procedures. I'm reading the contributor's guide now...
Did I get everything right this time?
Comment #4
drummCode looks good. I can see how this might be a problem. +1
(I didn't try running with this patch)
Comment #5
fmann commentedI applied the patch and it fixed the problem for me. Thank you much!
This is the first bug I've submitted here and I want to make sure the fix gets merged in.
Should I change the status of this post to fixed?
Comment #6
ezheidtmann commentedFloydM: The status is changed to fixed only when/if the patch is applied.
Comment #7
fmann commentedGood deal. Thank you, clydefrog (and thank you HellRaider for the patch).
Comment #8
Stefan Nagtegaal commentedI can confirm that this patch works as it should, had the same errors FloydM when trying to put a block on the frontpage of my site..
Good catch HellRaider! :-)
Comment #9
dries commentedHellRaiser, thanks for attaching the patch. You got it right. :) Committed to HEAD and DRUPAL-4-6. Thanks.
Comment #10
dries commentedComment #11
(not verified) commentedComment #12
(not verified) commentedComment #13
ax commentedComment #14
(not verified) commentedComment #15
(not verified) commentedComment #16
(not verified) commented