search

MITLibraries / oastats-ui

The User Interface for the MIT Libraries Open Access Statistics Project
0 stars 2 forks source link

Changes from require_once to require for credentials #135

Closed matt-bernhardt closed 9 years ago

matt-bernhardt commented 9 years ago

Using require_once caused authentication errors for administrators in certain specific cases. Tagging @JPrevost for review...

JPrevost commented 9 years ago

:+1: but I think we've identified @gravesm as the actual person who should review these :)

JPrevost commented 9 years ago

FWIW, require_once is all over the place: https://github.com/MITLibraries/oastats-ui/search?utf8=✓&q=require_once

Maybe it's fine in the other contexts?