Closed (fixed)
Project:
Feedback
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
18 Jan 2010 at 16:46 UTC
Updated:
3 Jan 2014 at 01:08 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
sunMost likely, your site's theme is interfering with the content markup. Try with one of the stock core themes and if that works, debug your theme.
Comment #2
ragavendra_bn commentedThanks sun for the aid at the right time. As you said, it has got to do with the theme itself I to suppose. I got the below error in the log. Not sure, as to how to go about it.
Message Parameter 1 to theme_feedback_admin_view_form() expected to be a reference, value given in /home/fashion/public_html/store/includes/theme.inc on line 617.
Note: I am not having any issues with the same code on my localhost, which has php-5.2.9-2 version.
Recently the PHP on the hosting server was updated to 5.3.1. May be this could be it.
Comment #3
sunThanks, now we can do something about this.
Comment #4
ragavendra_bn commentedYup. You are right Sir............
Only need to remove the & before &$form.
Patched line# 66 on feedback.admin.inc as below
ec_customer.theme.inc line 53
function theme_feedback_admin_view_form(&$form) {
+ function theme_feedback_admin_view_form($form) {
Able to see the messages as of new. Not sure of any more updates that needs to be done.
Comment #5
oriol_e9gtagged
Comment #6
berdirYep, removing the & is correct. The attached patch does this too.
Comment #7
sunThanks for reporting, reviewing, and testing! Committed to all branches.
A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.