Add utilities for the javascript message catalog.

Review Request #4451 — Created Aug. 13, 2013 and submitted

Information

Djblets
master

Reviewers

Add utilities for the javascript message catalog.

Django has a view for serving a javascript implementation of gettext and a
message catalog for the desired language. This view recomputes everything every
time it's fetched, which is kind of terrible. This change adds some utilities
for caching the data in a way which works like our old media serial did.
Used this in conjunction with some mocked-up javascript i18n files for
different languages and saw it generate and use good cache keys.
Description From Last Updated

We should have a try/except here, and log any failed attempts. ImportErrors can easily masquerade as other, confusing failures.

chipx86chipx86

Blank line before.

chipx86chipx86

""" on the next line.

chipx86chipx86

This should be at the top of the function, with a blank line after.

chipx86chipx86
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
        djblets/util/views.py
        djblets/util/misc.py
      Ignored Files:
    
    
  2. 
      
reviewbot
  1. This is a review from Review Bot.
      Tool: Pyflakes
      Processed Files:
        djblets/util/views.py
        djblets/util/misc.py
      Ignored Files:
    
    
  2. 
      
chipx86
  1. 
      
  2. djblets/util/misc.py (Diff revision 1)
     
     
    Show all issues
    We should have a try/except here, and log any failed attempts. ImportErrors can easily masquerade as other, confusing failures.
  3. djblets/util/misc.py (Diff revision 1)
     
     
    Show all issues
    Blank line before.
  4. djblets/util/views.py (Diff revision 1)
     
     
    Show all issues
    """ on the next line.
  5. 
      
david
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
        djblets/util/views.py
        djblets/util/misc.py
      Ignored Files:
    
    
  2. 
      
reviewbot
  1. This is a review from Review Bot.
      Tool: Pyflakes
      Processed Files:
        djblets/util/views.py
        djblets/util/misc.py
      Ignored Files:
    
    
  2. 
      
david
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
        djblets/util/views.py
        djblets/util/misc.py
      Ignored Files:
    
    
  2. 
      
reviewbot
  1. This is a review from Review Bot.
      Tool: Pyflakes
      Processed Files:
        djblets/util/views.py
        djblets/util/misc.py
      Ignored Files:
    
    
  2. 
      
chipx86
  1. 
      
  2. djblets/util/views.py (Diff revision 3)
     
     
    Show all issues
    This should be at the top of the function, with a blank line after.
  3. 
      
david
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
        djblets/util/views.py
        djblets/util/misc.py
      Ignored Files:
    
    
  2. 
      
reviewbot
  1. This is a review from Review Bot.
      Tool: Pyflakes
      Processed Files:
        djblets/util/views.py
        djblets/util/misc.py
      Ignored Files:
    
    
  2. 
      
chipx86
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to master (cac534e).
Loading...