craigk5n / webcalendar

WebCalendar is a PHP application used to maintain a calendar for a single user or an intranet group of users. It can also be configured as an event calendar.
http://www.k5n.us/webcalendar.php
GNU General Public License v2.0
149 stars 71 forks source link

convert include|require to require_once #358

Closed bbannon closed 11 months ago

bbannon commented 1 year ago

Craig, you started to do this a few weeks ago. Did you change your mind? Did you remenber why you converted them all back years ago?

craigk5n commented 1 year ago

I believe require is preferable since it will abort and exit if not found while include will keep going.

bbannon commented 1 year ago

I like require_once so we don't try to load things more than once.

I think I converted all the include / include_once / require Including a few sneaky ones hidden deep in the code by themselves.

Bruce

On Sun, Apr 23, 2023 at 9:21 AM Craig Knudsen @.***> wrote:

I believe require is preferable since it will abort and exit if not found while include will keep going.

— Reply to this email directly, view it on GitHub https://github.com/craigk5n/webcalendar/pull/358#issuecomment-1519091902, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHJ7GLTTBXU6S76PPKFLAVTXCVCIJANCNFSM6AAAAAAW6QN374 . You are receiving this because you authored the thread.Message ID: @.***>