openedoo / module_employee

MIT License
0 stars 0 forks source link

Advice for placing login_required in better directory #1

Open rendiya opened 7 years ago

rendiya commented 7 years ago

def login_required(f):
    @wraps(f)
    def wrap(*args, **kwargs):
        session.permanent = True
        try:
            if session['username'] is False:
                flash('You must login first!')
                return redirect(url_for('module_employee.login'))
            return f(*args, **kwargs)
        except KeyError:
            flash('Your session is timeout!')
            return redirect(url_for('module_employee.login'))
return wrap

klo bisa ini di taruh di model...

ghost commented 7 years ago

Menurut saya login_required jangan ditempatkan di model, karena model seharusnya hanya berisi operasi-operasi yang berhubungan dengan database.

Setelah saya mempelajari flask project structure dari google, saya mendapatkan ini http://flask.pocoo.org/docs/0.12/patterns/viewdecorators/ . Bagaimana kalau saya membuat folder lagi bernama views yang nanti akan berisi helper atau fungsi pembantu yang berhubungan di bagian view, termasuk login_required ini?.

Jadi nanti foldernya seperti ini

-- forms
-- models
-- views
-- statics
-- templates