sproctor / php-calendar

PHP-Calendar
http://www.php-calendar.org/
Apache License 2.0
140 stars 67 forks source link

don't require install directory to be removed #4

Closed sproctor closed 11 years ago

sproctor commented 11 years ago

instead of requiring the install directory to be removed, we can make the install script inoperable when there is a config.php with content.

sabas commented 11 years ago

The other thing, in setup.php lines from 38 should reflect something like this

if(!defined("PHPC_DEBUG") && file_exists("$phpc_root_path/install.php")) {

echo <<<END
<div class="ui-widget">
    <div class="ui-state-error ui-corner-all" style="padding: 0 .7em;">
        <p><span class="ui-icon ui-icon-alert" style="float: left; margin-right: .3em;"></span>
        <strong>Alert:</strong> You have to remove the install.php file!!</p>
    </div>
</div>
END;
    //display_error(_("You must remove the install script to start using the calendar. If you see this error but you must install the database, goto <a href=\"install.php\">install.php</a>"));
}

No more soft error, but a big alert on top of the calendar.

Must be formatted nicely :D

sabas commented 11 years ago

15 works, but index.php goes in soft error till we don't remove install.php

sproctor commented 11 years ago

Brain is apparently mush today. Just committed that fix.