cweiske/phorkie — pull request #11 by jnovack (2012-09-25 23:07:41+00:00)
Title: Fix SetupCheck / Indexing

I wanted to get your input on the changes made on ```src/phorkie/SetupCheck.php```:

* 776fadd - I filed a issue with the author of PHP-Markdown (https://github.com/michelf/php-markdown/pull/46#issuecomment-8869844) because it does not confirm to PEAR naming conventions for the class and filename. It does not autoload. Therefore, I'm removing it temporarily from the index check.

* 494f8ff - Added (cursory) checking for a valid elastic search url. You MUST use a namespace (e.g ```/phorkie/```) on the url, you cannot use the root directory (i.e. ```/```). This isn't obvious in the README, and the cause of my elasticsearch woes.

* 12f8fc8 - Updated README.

Once PHP-Markdown fixes its issue, I propose making ```scripts/index.php``` a more generic "run this script after setup" rather than a "elasticsearch index script", which should be run inbetween step 5 and 6 during installation.