Django

Code

Changeset 2815

Show
Ignore:
Timestamp:
05/02/06 13:44:22 (2 years ago)
Author:
adrian
Message:

Fixed #1059 -- Fixed off-by-one line number in heading of debug error pages.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • django/trunk/django/views/debug.py

    r2809 r2815  
    101101            'filename': filename, 
    102102            'function': function, 
    103             'lineno': lineno, 
     103            'lineno': lineno + 1, 
    104104            'vars': tb.tb_frame.f_locals.items(), 
    105105            'id': id(tb), 
    …… 
    107107            'context_line': context_line, 
    108108            'post_context': post_context, 
    109             'pre_context_lineno': pre_context_lineno, 
     109            'pre_context_lineno': pre_context_lineno + 1, 
    110110        }) 
    111111        tb = tb.tb_next 
    …… 
    361361            <div class="context" id="c{{ frame.id }}"> 
    362362              {% if frame.pre_context %} 
    363                 <ol start="{{ frame.pre_context_lineno|add:"1" }}" class="pre-context" id="pre{{ frame.id }}">{% for line in frame.pre_context %}<li onclick="toggle('pre{{ frame.id }}', 'post{{ frame.id }}')">{{ line|escape }}</li>{% endfor %}</ol> 
     363                <ol start="{{ frame.pre_context_lineno }}" class="pre-context" id="pre{{ frame.id }}">{% for line in frame.pre_context %}<li onclick="toggle('pre{{ frame.id }}', 'post{{ frame.id }}')">{{ line|escape }}</li>{% endfor %}</ol> 
    364364              {% endif %} 
    365               <ol start="{{ frame.lineno|add:"1" }}" class="context-line"><li onclick="toggle('pre{{ frame.id }}', 'post{{ frame.id }}')">{{ frame.context_line|escape }} <span>...</span></li></ol> 
     365              <ol start="{{ frame.lineno }}" class="context-line"><li onclick="toggle('pre{{ frame.id }}', 'post{{ frame.id }}')">{{ frame.context_line|escape }} <span>...</span></li></ol> 
    366366              {% if frame.post_context %} 
    367                 <ol start='{{ frame.lineno|add:"2" }}' class="post-context" id="post{{ frame.id }}">{% for line in frame.post_context %}<li onclick="toggle('pre{{ frame.id }}', 'post{{ frame.id }}')">{{ line|escape }}</li>{% endfor %}</ol> 
     367                <ol start='{{ frame.lineno|add:"1" }}' class="post-context" id="post{{ frame.id }}">{% for line in frame.post_context %}<li onclick="toggle('pre{{ frame.id }}', 'post{{ frame.id }}')">{{ line|escape }}</li>{% endfor %}</ol> 
    368368              {% endif %} 
    369369            </div> 
    …… 
    405405  File "{{ frame.filename }}" in {{ frame.function }}<br/> 
    406406  {% if frame.context_line %} 
    407     &nbsp;&nbsp;{{ frame.lineno|add:"1" }}. {{ frame.context_line|escape }}<br/> 
     407    &nbsp;&nbsp;{{ frame.lineno }}. {{ frame.context_line|escape }}<br/> 
    408408  {% endif %} 
    409409{% endfor %}<br/>