Closed (fixed)
Project:
Drupal core
Version:
6.x-dev
Component:
blog.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
4 Aug 2008 at 10:56 UTC
Updated:
28 Sep 2009 at 15:10 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
wanjee commentedSame problem here using Drupal 6.9.
I think this should be fixed as it is the only place where 'edit own blog' is specified.
The only required change is on line 61 of blog.pages.inc file.
if (user_access('edit own blog')) {should become
if (user_access('edit own blog entries')) {It is stated in comment of system_update_6045() function that
Hope to hear from someone for a positive feedback or a explanation on how it is like that.
Regards,
Seb
Comment #2
wanjee commentedSo here is the patch to allow roles with "edit own blog entries" permission to get the link in top of the /blogs page.
Hope it's ok as it is my first one.
Comment #3
Anonymous (not verified) commentedWhile the patch is good we need to roll it against CVS head first. Can you please reroll the patch for D7?
Comment #4
wanjee commentedHere is the new version.
The fix is different as the correct permission in D7 seems to be 'edit own blog content' instead of 'edit own blog entries' so I have made this little change in new patch.
Regards
Comment #5
dave reidComment #7
wanjee commentedI don't see any syntax error in the patch. Is this a test error ?
Regards
Comment #8
dave reidHmm...there was an extra newline at the end of the patch file. Let's try this one.
Comment #10
dave reidHmm...I rolled a new patch myself and fixed the permission to use the correct one 'create blog content', since the link that is used is for creating new blog content, and not editing it.
Comment #11
wanjee commentedSure you are right, 'create blog content' is far better.
So I guess 'create blog entries' will be the correct permission in 6.x
Thanks for reviewing this.
Regards,
Seb
Comment #13
jody lynnRetest.
Comment #14
dave reidThis time with an added assertion in blog.test that I've confirmed fails before the patch, and passes with the patch. Yay! Let's get this reviewed and fixed!
Comment #15
jody lynnLooks good to me.
Comment #16
dries commentedComimtted to CVS HEAD. Thanks.
Comment #17
jody lynnNeeds a re-roll for the backport, without the test.
Comment #18
wanjee commentedRe-roll for Drupal 6.
Hope it's ok.
Regards,
Wanjee
Comment #19
jody lynnLooks good.
Comment #20
gábor hojtsyYes, for Drupal 6, the good permission is "create blog entries". From system.install's update docs:
Committed to Drupal 6, thanks!