integration with site_map.module

I'm not that familiar with the sitemap module - gonna go download it now though. So would you mind providing more details on how you imagine it should work?

Sitemap already picks up the categories. It doesn't look to me like it (Sitemap) was ever really intended to drill down below a page level (certainly the search engines wouldn't want it to). But, since the questions/answers are individual nodes, I would see this as a request for Sitemap, not FAQ.

nancyw, it is actually a feature request for both modules, first for the faq module, and following on from that, for the site_map module.

The faq module needs to provide function(s) that the site_map module can use to list the FAQ nodes. That way the code required to support the faq nodes in the site_map module can be kept to a minimum, which makes it easier to maintain, since they shouldn't have to do a new release of their module every time I modify the faq module. So first the faq module needs to have additional functions added to it. Once that's done, the maintainer of the site_map module should only need to make a small change to call the necessary function(s) and display the data returned.

NikLP - I'd like to hear more about what you would like displayed. At the moment, I'm thinking of a function that will return a list of the FAQ questions and links to their nodes. The questions would be returned in the order as determined by their configured weights. However, if categories are used, then it may need to return the list of questions per category, or maybe it should just return links to the categorised faq pages.

Regards,
Stella

Added two new functions to the module in CVS:

* faq_get_faq_list() - returns a list of weighted FAQ nodes. However if categories are enabled, it just returns the FAQ categories.
* faq_get_terms() - returns a list of FAQ categories.

These will be included in the next release.

The maintainer said he'd be happy to add it. However I have to release it first, but a release is dependant on another issue being fixed.

FAQ part released in faq-5.x-2.3.

Two new functions have been added which can be used by the site_map module:

• faq_get_faq_list() - returns a list of weighted FAQ nodes. However if categories are enabled, it just returns the FAQ categories.
• faq_get_terms() - returns a list of FAQ categories.

Cheers,
Stella

I have committed a solution to HEAD but I have not tested it. If you use the FAQ module please try it out and report back any issues here.

I just committed a patch that adds a hook_site_map() to the site map module. I believe something like this in the faq module should work. This I a more flexibel solution than my first attempt above.

/**
 * Implementation of hook_site_map().
 */
function faq_site_map() {
  $title = variable_get('faq_title', t('Frequently Asked Questions'));
  $output = faq_get_faq_list();
  return theme('box', $title, $output);
}

Would it be possible to add a checkbox to the sitemap configuration screen that enables / disables the FAQ display in the generated sitemap?

Stella

I've added the hook to the faq module in CVS.

Hook added to faq module now released in faq-5.x-2.4.

Would it be possible to add a checkbox to the sitemap configuration screen that enables / disables the FAQ display in the generated sitemap?

Cheers,
Stella

Flag appears to be there now. Thanks.

Cheers,
Stella