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 trialJosh Keenan
20,315 Pointsfollow view
Can't see the problem, I will include another attempt I made as well:
try:
to_user = models.User.get(models.User.user_id**user_id)#case insensitve search **
except models.DoesNotExist:
pass
else:
try:
models.Relationship.create(
from_user=g.user._get_current_object(),
to_user=to_user
)
except models.IntegrityError:
pass
else:
flash("You've followed {}!".format(to_user.user_id), "success")
return redirect(url_for('index'))
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'
@app.route('/follow/<int:user_id>')
@login_required
def follow(user_id):
models.Relationship.create(
from_user=g.user._get_current_object(),
to_user=models.User.get(models.User.user_id)
)
return redirect(url_for('index'))
@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'))
4 Answers
Bart Bruneel
27,212 PointsSo here's the code that passes for me.
@app.route("/follow/<int:user_id>")
@login_required
def follow(user_id):
try:
to_user = models.User.get(models.User.id**user_id)#case insensitve search **
except models.DoesNotExist:
pass
else:
try:
models.Relationship.create(
from_user=g.user._get_current_object(),
to_user=to_user
)
except models.IntegrityError:
pass
else:
flash("You've followed {}!".format(to_user.id), "success")
return redirect(url_for('index'))
Bart Bruneel
27,212 PointsHello Josh,
Your first attempt is more or less ok if you add the
@app.route('/follow/<int:user_id>')
@login_required
def follow(user_id):
to the beginning. The only other thing that is wrong about your solution is that the id-attribute of the User-model is not called user_id but just id. So you will have to change user_id to id in the line where you create the to_user and in the line where you flash the message.
Josh Keenan
20,315 PointsNearly, but it doesn't create a relationship..
Josh Keenan
20,315 PointsIt's because in the scenario instead of a username attribute there was a user id attribute, so if you search for a username, it can't find one as it doesn't exist
dierkpolzin
7,380 Pointsdierkpolzin
7,380 PointsThis was a bit of crock...
Why does the flash message have to be "to_user.id shouldn't it be "to_user.username"
Than hung me up for an hour...
flash("You've followed {}!".format(to_user.id), "success") # works
flash("You've followed {}!".format(to_user.username), "success") #fails..
pretty damn picky and wrong if you ask me.