Skip to content
This repository has been archived by the owner on Feb 16, 2020. It is now read-only.

Receiving the following errors upon install #1

Open
akeslo opened this issue Aug 2, 2017 · 4 comments
Open

Receiving the following errors upon install #1

akeslo opened this issue Aug 2, 2017 · 4 comments

Comments

@akeslo
Copy link

akeslo commented Aug 2, 2017

image

image

@volomike
Copy link

I am getting the same problems.

@volomike
Copy link

I grepped the source. The failed to load keyfile comes from php/newsletter/config.php. The failed to enumerate {directory}, no such directory comes from php/newsletter/common/common.php. Still digging...

@volomike
Copy link

By the way, you don't need to end scripts with ?> if you're not emitting any HTML in them. The official PHP website also mentions this as a proper practice, too, although I can't recall the link for that right now. There's another reason why this is important, too. Just add a space after one of those ?> statements and sometimes you can trigger strange bugs to happen that can be maddening to find.

@volomike
Copy link

The whole project needs work. You can go a long way if you would add this when doing your development, and then comment it out before posting into production:

error_reporting(E_ALL);
ini_set('display_errors','On');

I'm finding several issues. Missing file by file paths. I had to create a /php/newsletter/settings/server.json and /php/newsletter/settings/keyfile, although I had no clue what to put in them yet because your docs don't cover this. I also had to add dirname(__FILE__) (newer PHP can utilize __DIR__ before the paths so that the file_get_contents() could know what path you're talking about in your code.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants