Welcome to the Treehouse Community
Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.
Looking to learn something new?
Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.
Start your free trial
Nitesh Kumar
1,055 PointsBummer! Didn't find an H1 with 404 in it.
I am getting this error continuously. I have done everything as it is asked in the question. I don't know what could possibly be wrong with my code. Here is what i have to do -
Challenge Task 3 of 3
Now make not_found render the "404.html" template. Add an <h1> to the template with the 404 error code in it. Add a message to the template, if you want.
And this is what i have done -
import datetime
from flask import Flask, g, render_template, flash, redirect, url_for, abort
from flask.ext.bcrypt import check_password_hash
from flask.ext.login import LoginManager, login_user, current_user, login_required, logout_user
import forms
import models
app = Flask(__name__)
app.secret_key = 'this is our super secret key. do not share it with anyone!'
login_manager = LoginManager()
login_manager.init_app(app)
login_manager.login_view = 'login'
@login_manager.user_loader
def load_user(userid):
try:
return models.User.select().where(
models.User.id == int(userid)
).get()
except models.DoesNotExist:
return None
@app.before_request
def before_request():
g.db = models.DATABASE
g.db.connect()
g.user = current_user
@app.after_request
def after_request(response):
g.db.close()
return response
@app.route('/register', methods=('GET', 'POST'))
def register():
form = forms.SignUpInForm()
if form.validate_on_submit():
models.User.new(
email=form.email.data,
password=form.password.data
)
flash("Thanks for registering!")
return render_template('register.html', form=form)
@app.route('/login', methods=('GET', 'POST'))
def login():
form = forms.SignUpInForm()
if form.validate_on_submit():
try:
user = models.User.get(
models.User.email == form.email.data
)
if check_password_hash(user.password, form.password.data):
login_user(user)
flash("You're now logged in!")
else:
flash("No user with that email/password combo")
except models.DoesNotExist:
flash("No user with that email/password combo")
return render_template('register.html', form=form)
@app.route('/secret')
@login_required
def secret():
return "I should only be visible to logged-in users"
@app.route('/logout')
def logout():
logout_user()
return redirect(url_for('login'))
@app.route('/')
def index():
return render_template('index.html')
@app.route('/order', methods=('GET', 'POST'))
def order_lunch():
form = forms.LunchOrderForm()
if form.validate_on_submit():
models.LunchOrder.create(
user=g.user._get_current_object(),
date=form.date.data,
order=form.order.data.strip()
)
return render_template('lunch.html', form=form)
@app.route('/today')
@login_required
def today():
order = models.LunchOrder.select().where(
models.LunchOrder.date == datetime.date.today() &
models.LunchOrder.user == g.user._get_current_object()
).get()
return render_template('today.html', order=order)
@app.route('/cancel_order/<int:order_id>')
@login_required
def cancel_order(order_id):
try:
order = models.LunchOrder.select().where(
id=order_id,
user=g.user._get_current_object()
).get()
except models.DoesNotExist:
pass
else:
order.delete_instance()
return redirect(url_for('index'))
@app.route('/follow/<int:user_id>')
@login_required
def follow(user_id):
try:
user = models.User.get(
models.User.id == user_id
)
models.Relationship.create(
from_user=g.user._get_current_object(),
to_user=user
)
except (models.DoesNotExist, models.IntegrityError):
pass
return redirect(url_for('index'))
@app.route('/unfollow/<int:user_id>')
@login_required
def unfollow(user_id):
try:
user = models.User.get(
models.User.id == user_id
)
models.Relationship.get(
models.Relationship.from_user==g.user._get_current_object(),
models.Relationship.to_user==user
).delete_instance()
except (models.DoesNotExist, models.IntegrityError):
pass
return redirect(url_for('index'))
@app.route('/profile/<int:user_id>')
def profile(user_id):
user = models.User.select().where(
models.User.id == user_id
).get()
return render_template('profile.html', user=user)
@app.errorhandler(404)
def not_found(error):
return "404", 404
return render_template('404.html'), 404
And here is my 404.html
{% extends "layout.html" %}
{% block content %}
<h1> 404 </h1>
<p>Wow, sorry, that page doesn't exist.</p>
{% endblock %}
4 Answers
Chris Freeman
Treehouse Moderator 68,468 PointsYou appear to have two return statements in:
@app.errorhandler(404)
def not_found(error):
return "404", 404
return render_template('404.html'), 404
Nitesh Kumar
1,055 PointsActually i can't remove that first return, because if i remove that than i was getting the new error "Task 2 seems not to pass". But i changed their order. I put render_template before.
Daniel Santos
34,969 PointsHey Nitesh, I understand your problem, but I will give you a hint, you should not use return statement one after another without any condition in between.
Nitesh Kumar
1,055 PointsThanks Daniel, I have solved the problem already, but yes you are right, it's really a bad coding practice to use multiple return statements together.
Iwan Engelbrecht
9,941 PointsStill don't get this??
Chris Freeman
Treehouse Moderator 68,468 PointsIwan, please feel free to start a new post with your code and we'll figure what you need! Tag me in the post or comments if you wish!