Documentation for locale_selector callback for i18n module is broken about webapp-improved

[GoogleCodeExporter]

Hi,
This change was made a while ago to be consistent with how callables are set in 
other modules. 

Version 1.8 - June 26, 2011
===========================
- webap2_extras.i18n: the functions passed to I18nStore.set_locale_selector()
  and I18nStore.set_timezone_selector() now receive (store, request) instead of
  (request, store), for consistency with the router custom callables.

So the current i18n module actually has inverted parameters.

Sorry if this caused you trouble... I've been taking care to not add 
incompatibilities like this one. Do you suggest making the inverted parameters 
backwards compatible or can I close this ticket?

Original comment by rodrigo.moraes on 10 Aug 2011 at 4:26

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

Hi Rogrigo,

I just started using webapp2 just recently so I wasn't quite up to date with 
recent changes :) Anyway, I don't suggest anything, I was merely pointing out 
the inconsistency in the docs as I'm sure there will be others making the same 
mistake I made. 

Original comment by [email protected] on 10 Aug 2011 at 4:36

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

The docs are up-to-date to latest release. You are using a pre-1.8 version, 
when the arguments were indeed inverted.

Original comment by rodrigo.moraes on 10 Aug 2011 at 4:49

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

No, I am using webapp2-2.2.3.zip. If store is the first argument to 
locale_selector the docs are definately not up to date. 

Original comment by [email protected] on 10 Aug 2011 at 5:18

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

Hmmm... Maybe you are reading a cached version? This is what I see there:

(sorry, copy & paste mess)

set_locale_selector(func)[source]
Sets the function that defines the locale for a request.
Parameters: func – A callable that receives (store, request) and returns the 
locale for a request.

set_timezone_selector(func)[source]
Sets the function that defines the timezone for a request.
Parameters: func – A callable that receives (store, request) and returns the 
timezone for a request.

Original comment by rodrigo.moraes on 10 Aug 2011 at 5:27

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

Look in the parameter list for webapp2_extras.i18n.default_config at the top
of the page :)

/F

Original comment by [email protected] on 10 Aug 2011 at 5:33

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

Ah. Sorry for all the noise. Totally missed that one. Updating it.

Original comment by rodrigo.moraes on 10 Aug 2011 at 6:00

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

No worries. Thanks for making webapp2 so good. :-)

Cheers / Fredrik

locale_selector

Original comment by [email protected] on 10 Aug 2011 at 6:03

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

This issue was closed by revision 71241f3a07c9.

Original comment by rodrigo.moraes on 10 Aug 2011 at 6:04

• Changed state: Fixed
• Added labels: ****
• Removed labels: ****