e107inc / e107

e107 Bootstrap CMS (Content Management System) v2 with PHP, MySQL, HTML5, jQuery and Twitter Bootstrap. Issue Discussion Room: https://gitter.im/e107inc/e107
https://e107.org
GNU General Public License v3.0
318 stars 212 forks source link

Fix #5052: Guard potentially undefined `e_PAGE` constant #5053

Closed Deltik closed 10 months ago

Deltik commented 11 months ago

Motivation and Context

Fixes: https://github.com/e107inc/e107/issues/5052

Description

Guard potentially undefined e_PAGE constant

How Has This Been Tested?

By manually running through the "How to Reproduce" steps in https://github.com/e107inc/e107/issues/5052

Types of Changes

Checklist

codeclimate[bot] commented 11 months ago

Code Climate has analyzed commit 68b2401b and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Bug Risk 1

The test coverage on the diff in this pull request is 0.0% (80% is the threshold).

This pull request will bring the total coverage in the repository to 34.1% (-0.8% change).

View more on Code Climate.

CaMer0n commented 10 months ago

Thank you. Closed in favor of https://github.com/e107inc/e107/commit/5ff319cd5c55e0a5a90c33af5d713b02037e585d