Closed (fixed)
Project:
Frequently Asked Questions
Version:
master
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
9 Jun 2006 at 07:53 UTC
Updated:
9 Mar 2007 at 11:35 UTC
Jump to comment: Most recent file
Attached is a patch that updates the FAQ module to 4.7 compatibility. Note that I've included a _install hook, so faq.mysql should be unnecessary. Also, I approached fixing FAQ for mod_rewrite than http://drupal.org/node/67397 which uses the l() function. Since there are link anchors, I used the url() function instead, which means keeping some of the HTML in the module intact.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | drupal.faq.module | 9.37 KB | halkeye |
| faq.47.patch.txt | 3.8 KB | sillygwailo |
Comments
Comment #1
sillygwailoComment #2
bomarmonk commentedI am getting some warnings after applying this patch and using this module under 4.7:
warning: include_once(./modules/faq/faq.module) [function.include-once]: failed to open stream: Permission denied in C:\Program Files\xampp\htdocs\mysite\includes\bootstrap.inc on line 487.
warning: include_once() [function.include]: Failed opening './modules/faq/faq.module' for inclusion (include_path='.;C:/Program Files/xampp/htdocs/mysite/modules/civicrm/;C:/Program Files/xampp/htdocs/alpinecounty/modules/civicrm/\packages;.;C:\Program Files\xampp\php\pear\') in C:\Program Files\xampp\htdocs\mysite\includes\bootstrap.inc on line 487.
Thanks for the patch!
Comment #3
halkeye commented4.7 updates
fixed more of the links than the previous patches..
Comment #4
pietrocap commentedIn function theme_faq_highlights in my opinion there is an incorrect use of parameters for l() function. Using parameters in this way anchors are passed as querystring. Here is my patch:
Comment #5
pal_ur commentedPlease notice, in faq.module at aboute line 260-262:
have to be corrected to
to able to edit the answer.
Comment #6
wmostrey commentedI noticed with this patch for 4.7 that when I 'create content', the 'faq' option doesn't appear in the menu. It appears in the list in the content field with the explenation, but it doesn't appear on the left.. It has the hook_node_info so I'm not sure what the problem is. Do other people experience this too?
All that seems ok.. Anybody an idea?
Comment #7
alexkb commentedHey halkeye, thanks for that patch, it works great. Though for anyone else looking to use it, make sure you apply that patch on the version that hasn't already been patched by Richard's patch.
Anyway, in all patches, I still came across the following error:
Unknown column 'n.teaser' in 'field list' query: SELECT n.nid, n.title, n.teaser FROM node n WHERE n.type = "faq" AND n.status = 1 ORDER BY n.created DESC LIMIT 5 in /######/includes/database.mysql.inc on line 121.To fix this, I simply removed the references to "teaser" in the theme_faq_highlight function.
Also, the comment `wmostrey` made about it showing up in the menu, I think this has to do the faq_form function, but I tried adding an entry 'add/node/faq' for which it still failed to show up in the menu. I guess we will have to make do for now.
Comment #8
stella commentedA drupal 4.7 version of this module has been released - version 4.7.x-1.0-1.
A drupal 5 version now also exists and it is substantially different from the previous versions - 5.x-1.0.
Cheers,
Stella
Comment #9
stella commented