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

Comments

tom robert’s picture

tom robert commented
Status: Active » Needs review
StatusFileSize
new site_map-7.x-2.x-alpha-1545422-0.patch674 bytes
new 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 commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new 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 commented

Works perfectly! Thanks

darrell_ulm’s picture

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 🇨🇦
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 commented
StatusFileSize
new 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 🇨🇦
commented

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