Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1015)

Issue 101650043: Fixed the traceback on trying to access non-readable item as per issue #444 (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 4 months ago by ajgupta93
Modified:
10 years, 3 months ago
Reviewers:
RogerHaase
Visibility:
Public.

Description

Fixed the traceback on trying to access non-readable item

Patch Set 1 #

Total comments: 2

Patch Set 2 : Used the old function for 403 error #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -0 lines) Patch
M MoinMoin/themes/basic/templates/layout.html View 1 1 chunk +2 lines, -0 lines 1 comment Download

Messages

Total messages: 3
RogerHaase
https://codereview.appspot.com/101650043/diff/1/MoinMoin/apps/frontend/views.py File MoinMoin/apps/frontend/views.py (right): https://codereview.appspot.com/101650043/diff/1/MoinMoin/apps/frontend/views.py#newcode2308 MoinMoin/apps/frontend/views.py:2308: title=title, description=description) hmm, on second thought, I don't think ...
10 years, 4 months ago (2014-07-02 14:55:19 UTC) #1
RogerHaase
https://codereview.appspot.com/101650043/diff/1/MoinMoin/apps/frontend/views.py File MoinMoin/apps/frontend/views.py (right): https://codereview.appspot.com/101650043/diff/1/MoinMoin/apps/frontend/views.py#newcode2308 MoinMoin/apps/frontend/views.py:2308: title=title, description=description) On 2014/07/02 14:55:19, RogerHaase wrote: > hmm, ...
10 years, 4 months ago (2014-07-02 18:26:36 UTC) #2
RogerHaase
10 years, 4 months ago (2014-07-04 17:03:44 UTC) #3
you should edit the issue text to include the issue number. Try to make issue
text usable for eventual commit message.

https://codereview.appspot.com/101650043/diff/20001/MoinMoin/themes/basic/tem...
File MoinMoin/themes/basic/templates/layout.html (right):

https://codereview.appspot.com/101650043/diff/20001/MoinMoin/themes/basic/tem...
MoinMoin/themes/basic/templates/layout.html:8: {% set user_actions,
item_navigation, item_actions = theme_supp.get_local_panel(item_name) %}
indent for clarity
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b