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 trialjamie tam
2,534 PointsAdd a view named follow with a route of "/follow/<int:user_id>". It should be login_required. In the view, create a new
Add a view named follow with a route of "/follow/<int:user_id>". It should be login_required. In the view, create a new Relationship with the current user as the from_user and the user with the provided ID as the to_user. Return a redirect to index.
What is wrong with my code?
import datetime
from flask import Flask, g, render_template, flash, redirect, url_for
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:
models.Relationship.create(
from_user=g.user._get_current_object(),
to_user=models.User.get(models.User.user_id**user_id)
)
except:
pass
return redirect(url_for('index'))
5 Answers
Allison Schaaf
33,322 PointsThis code passed the challenge:
@app.route("/follow/<int:user_id>")
@login_required
def follow(user_id):
to_user = models.User.get(models.User.id**user_id)
models.Relationship.create(
from_user=g.user._get_current_object(),
to_user=to_user
)
return redirect(url_for("index"))
Bashar Ghadanfar
28,309 PointsThe problem is in the way you assigned the user_id to the id of the User model.
This should fix it:
to_user = models.User.get(models.User.id==user_id)
Andrii Gorokh
14,809 Pointscan someone explain this part?
Mel Rumsey
Treehouse ModeratorI think the issue everyone was having was that they were putting an underscore in user_id. The instructions even had it "/follow/<int:user_id>", once I changed it to userid it worked :)
Wayne Smith
16,158 PointsJust did the challenge, it was hard to know the reason I was failing was because
to_user = models.User.get(models.User.user_id==user_id)
was incorrect. The feedback given on failure was merely "Bummer try again" which was completely unhelpful in trying to find the root cause of why it was failing, But after reading the comments changing to
to_user = models.User.get(models.User.id==user_id)
passed the challenge
Ivan Rzhanoi
16,596 PointsKenneth Love, please help us. My code is:
@app.route("/follow/<int:user_id>")
@login_required
def follow(user_id):
to_user = models.User.get(models.User.user_id**user_id)
models.Relationship.create(
from_user=g.user._get_current_object(),
to_user=to_user
)
return redirect(url_for("index"))
It just says "Try again!".
Kenneth Love
Treehouse Guest Teachermodels.User.user_id**user_id
is trying to raise models.User.user_id
by the power of user_id
. Pretty sure that's not what you want.
Radovan Valachovic
Python Web Development Techdegree Graduate 20,368 PointsHere: models.User.user_id*user_id >>> it suppose to be models.User.id*user_id (you are comparing it to ID field in the database)