468: diff viewer pagination counter showing string '{{ pages }}' rather than the total number of pages

jaybuf*******@gmai***** (Google Code) (Is this you? Claim this profile.)
April 18, 2008
On http://codereview.corp/r/7/diff/#index_header I'm seeing this:

"This diff has been split across {{ pages }} pages:"

diffviewer/changeindex.html contains this line:
 {% trans "This diff has been split across {{ pages }} pages:" %}

It looks like trans function doesn't expand {{ foo  }} variables.

See my comment here:
http://reviews.review-board.org/r/309/#last-review
chipx86
#1 chipx86
How old is your copy? That was fixed a while back.
#2 jaybuf*******@gmai***** (Google Code) (Is this you? Claim this profile.)
r1241

I did `svn diff diffviewer/changeindex.html` before submitting the bug report.  When it didn't output anything I 
assumed I had the latest version.  
#3 jaybuf*******@gmai***** (Google Code) (Is this you? Claim this profile.)
Oh, okay I did 

$ svn up diffviewer/changeindex.html 
U    diffviewer/changeindex.html
Updated to revision 1295.

and the bug went away.  I guess I need to learn how to use svn.

Thanks.
david
#4 david
  • +NotABug