Download & Extend
Advanced Forum » Issues

D7 Topic overview Error

Posted by Hydra on April 11, 2011 at 10:12pm
Project:Advanced Forum
Version:6.x-2.x-dev
Component:Views
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

When I enable the advanced forum modul, I receive this Message a dozen times.

Notice: Undefined property: view::$row_index in views_handler_field->get_render_tokens() (line 1198 of /var/kunden/webs/bcweb/spiderkingz/d7/sites/all/modules/views/handlers/views_handler_field.inc).

I'm using the actual 7.x-2.0-alpha1 of advanced_forum and the 7.x-3.x-dev views version.

Login or register to post comments

Comments

#1

Posted by dawehner on April 11, 2011 at 10:20pm
Priority:critical» normal
Status:active» needs review

Errors/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.

AttachmentSize
1124116-notice.patch 824 bytes

#2

Posted by Hydra on April 11, 2011 at 10:33pm
Priority:normal» critical
Status:needs review» active

patch from #1 dereine works for me! Thx

#3

Posted by fubhy on April 11, 2011 at 10:37pm
Status:active» reviewed & tested by the community

Changing status

#4

Posted by Hydra on April 11, 2011 at 10:40pm
Priority:critical» normal

#5

Posted by Hydra on April 11, 2011 at 10:43pm
Status:reviewed & tested by the community» closed (fixed)

-

#6

Posted by Hydra on April 11, 2011 at 10:45pm
Status:closed (fixed)» reviewed & tested by the community

#7

Posted by troky on April 12, 2011 at 1:24pm
Version:7.x-2.0-alpha1» 6.x-2.x-dev
Status:reviewed & tested by the community» patch (to be ported)

Ok, ok. Committed to 7.x-dev.

Thanks for the patch dereine.

#8

Posted by dawehner on April 12, 2011 at 2:47pm

@troky

Hydra is still learning drupal.org :(

#9

Posted by fubhy on April 12, 2011 at 2:51pm

Hydra = dereine's padavan :P

#10

Posted by jamescarvin on April 23, 2011 at 11:48am

You guys amaze me and we stand (or sit) indebted. I posted elsewhere on this as follows:

Notice: Undefined property: view::$row_index in views_handler_field->get_render_tokens() (line 1187 of /home/username/public_html/domainname.com/members/sites/all/modules/views/handlers/views_handler_field.inc).

This error message repeats quite a few times. I find it when browsing to the main page of the forum as any user. I will not want anyone seeing error messages other than admin. Of course, I don't want the error messages either.

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!

#11

Posted by dawehner on April 23, 2011 at 12:18pm

Advanced Forum 7.2x alpha1

You should better use the dev version of advanced forum, because it's fixed there.

#12

Posted by dawehner on April 23, 2011 at 12:19pm

Remove the most useless issue tag ever seen.

#13

Posted by mcdruid on June 19, 2011 at 12:25am
Status:patch (to be ported)» closed (fixed)

Thanks 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).

#14

Posted by NPC on June 21, 2011 at 11:44am

Thanks, the patch in #1 helped me on 7.x-2.0-alpha1.

#15

Posted by mcdruid on June 23, 2011 at 5:44pm
Status:closed (fixed)» fixed

oops - realised I should have been setting issues to "fixed" rather than "closed (fixed)"

#16

Posted by troky on June 23, 2011 at 5:58pm

Well, it is fixed and ported and there is no reason not to close it :)

#17

Posted by System Message on July 7, 2011 at 6:02pm
Status:fixed» closed (fixed)

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

nobody click here