Skip to content

Commit

Permalink
Upgrade - add other files that may need manual fix
Browse files Browse the repository at this point in the history
Tidy the 'Other included files' section, and mention:

app/config/config.php (how to fix security warning)
app/bootstrap.php (needed for upgrades from 1.6 or earlier)
  • Loading branch information
wturrell committed Oct 25, 2016
1 parent d6a4697 commit 7a5820f
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions installation/upgrade.html
Original file line number Diff line number Diff line change
Expand Up @@ -130,10 +130,14 @@ <h4 id="app_assets">Included asset files in public</h4>

<h4 id="app_others">Other included files</h4>
<p>
Other files that might need upgrading are the 'Oil' bootstrap script and the .htaccess file, and the files needed
by PHPUnit. If you haven't changed them, just replace them by the new version, to make sure you don't run into
issues when using them.
</p>
Other files may need upgrading. If you haven't changed them, just replace them with the new version, to make sure
you don't run into issues.</p>

<p><code>oil</code> - the 'Oil' bootstrap script, <code>.htaccess</code> and files needed by PHPUnit.</p>
<p> <code>app/config/config.php</code> - you may see an <em>"There is no security.output_filter defined"</em> error.<br/>
Keep your current config file, but copy over the "security" array from the latest version.</p>
<p> <code>app/bootstrap.php</code> - if you still have "Load in the Autoloader" (lines 3-6) these should be removed.<br/>
This is the cause of the <em>"Cannot declare class Fuel\Core\Autoloader, because the name is already in use"</em> error.</p>

<h3 id="tips">Using upstream repositories</h3>

Expand Down

0 comments on commit 7a5820f

Please sign in to comment.