search

cweiske / phorkie

Self-hosted pastebin software written in PHP. Pastes are editable, forkable, may have multiple files and are stored in git repositories.
https://cweiske.de/phorkie.htm
GNU Affero General Public License v3.0
216 stars 23 forks source link

Forkremote fix #15

Closed jnovack closed 12 years ago

jnovack commented 12 years ago

What was the line you said over in pull request #14 in comment 9199301?

It kinda seems to me as if you didn't even test it. When you switch E_NOTICE on in your php.ini's error_reporting setting, you'll see a bootstrap variable warning.

cweiske - 1, jnovack - 1 ;)

If fork-remote is submitted without a url, it's still set, but null.

This checks for both a null post and an incorrect url.

cweiske commented 12 years ago

I've merged the one commit and rewrote the second one since I didn't want more logic in www/fork-remote.php