The configuration options should be exposed to the variable module so that they can be configured per language. For example I might want to use one menu in one language and another one in another language.

CommentFileSizeAuthor
#8 1545422-8.patch853 bytesstefan.r
#2 site_map-7.x-1.x-1545422.patch674 bytesJax
#1 site_map-7.x-1.x-1545422-0.patch814 bytesTom Robert
#1 site_map-7.x-2.x-alpha-1545422-0.patch674 bytesTom Robert
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Tom Robert’s picture

Tom Robert CreditAttribution: Tom Robert commented
Status: Active » Needs review
FileSize
site_map-7.x-2.x-alpha-1545422-0.patch674 bytes
site_map-7.x-1.x-1545422-0.patch814 bytes

Added the menu site_map_show_menus variable to the hook_variable_info.
It will appear in the list Multilingual settings variables.

ref : http://drupal.org/node/1209500

Jax’s picture

Jax CreditAttribution: Jax commented
Status: Needs review » Reviewed & tested by the community
FileSize
site_map-7.x-1.x-1545422.patch674 bytes

Rerolled the 7.x-1.x patch as a p1 patch. Works.

bsztreha’s picture

bsztreha CreditAttribution: bsztreha commented

Works perfectly! Thanks

darrell_ulm’s picture

darrell_ulm CreditAttribution: darrell_ulm commented
Issue summary: View changes

@frjo marking this one, should we put it into dev for the next release? Thank you!

  • colan committed 5389d87 on 7.x-1.x authored by Tom Robert 
    Issue #1545422 by Tom Robert, Jax: Expose the menus to show to Variable.
colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
CreditAttribution: colan at Colan Schwartz Consulting commented
Status: Reviewed & tested by the community » Fixed

Thanks. Committed another reroll as this one didn't apply.

I've ignoring 2.x as that branch seems dead.

Status: Fixed » Closed (fixed)

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

stefan.r’s picture

stefan.r CreditAttribution: stefan.r commented
FileSize
1545422-8.patch853 bytes

Posting that reroll against 1.2 here as I need it for a makefile.

Could we do a new 7.x-1.3 release for the site_map module, even if a beta/rc?

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
CreditAttribution: colan at Colan Schwartz Consulting commented

Done. There is now a 7.x-1.3.