Closed (fixed)
Project:
Advanced Forum
Version:
6.x-2.x-dev
Component:
Views
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Apr 2011 at 22:12 UTC
Updated:
7 Jul 2011 at 18:02 UTC
Jump to comment: Most recent file
Comments
Comment #1
dawehnerErrors/warnings are actual problems. Notices should be fixed, but it's often it's just a bit like a codestyle issue.
See http://api.drupal.org/api/drupal/includes--bootstrap.inc/function/watchdog and http://www.ietf.org/rfc/rfc3164.txt
So update priority.
In d6 displaying notices would kill you!
This code now works more like the default table preprocess function.
You could use this in 6.x-1.x as well, because this is a function which is in views since a certain version.
Comment #2
hydra commentedpatch from #1 dereine works for me! Thx
Comment #3
fubhy commentedChanging status
Comment #4
hydra commentedComment #5
hydra commented-
Comment #6
hydra commentedComment #7
troky commentedOk, ok. Committed to 7.x-dev.
Thanks for the patch dereine.
Comment #8
dawehner@troky
Hydra is still learning drupal.org :(
Comment #9
fubhy commentedHydra = dereine's padavan :P
Comment #10
jamescarvin commentedYou guys amaze me and we stand (or sit) indebted. I posted elsewhere on this as follows:
Troky then directed me to this thread. Thanks! However, I'm a total newb and don't know what to do to fix the error.
I'm in D7 and have installed Advanced Forum 7.2x alpha1, CMA, ctools 7.x-1.x-dev, ACL-7.x-1.0-beta2
Maybe I should have avoided a dev version of ctools? Your help is always appreciated!
Comment #11
dawehnerYou should better use the dev version of advanced forum, because it's fixed there.
Comment #12
dawehnerRemove the most useless issue tag ever seen.
Comment #13
mcdruid commentedThanks dereine.
I tested the patch with views 6.x-2.12 and the render_fields method works fine.
Committed to the Advanced Forum 6.x-2.x branch (I had trouble attributing you as the patch author in git, sorry).
Comment #14
NPC commentedThanks, the patch in #1 helped me on 7.x-2.0-alpha1.
Comment #15
mcdruid commentedoops - realised I should have been setting issues to "fixed" rather than "closed (fixed)"
Comment #16
troky commentedWell, it is fixed and ported and there is no reason not to close it :)