Within the date_php4.inc file other files are included inside functions. When these functions are called many times this causes performance problems.

The enclosed patch simply moved the require_once statements to more appropriate places.

Before this I was getting maximum execution time exceeded messages on viewing a year calendar. While this is still very slow at least the page loads now.

CommentFileSizeAuthor
date_php4.inc_.moved_require.diff2.71 KBraintonr

Comments

karens’s picture

karens commented
Status: Needs review » Fixed

Php4 is pretty hard to work with and I'm not really supporting it any more, but I went ahead and committed this.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.