search

TheRosettaFoundation / SOLAS-Match

Self-managed translation project interface
www.TheRosettaFoundation.org
GNU Lesser General Public License v3.0
12 stars 8 forks source link

Add direcotries required by Smarty to the repository #1249

Closed aquilax closed 7 years ago

aquilax commented 7 years ago

Direcories ui/templating/cache/ and ui/templating/templates_compiled/ are required by Smarty or caching. This pull request adds the directories to the source control and excludes their content from commits.

Maybe it's a good idea to consider getting these out of the web root @alanbarrett ?

alanbarrett commented 7 years ago

@aquilax I am travelling so slower to respond.

It would be good to consider moving those two folders out of the document root. I am was not sure if the browser needed to access them directly? However it seems not as ui folder has .htaccess...

Order Deny,Allow Deny from all Allow from 127.0.0.1

Currently https://loc.csisdmz.ul.ie/wiki/index.php/Setup_of_Hetzner_Server_for_Trommons has those folders set as follows, not 100% sure why (give Apache write access?)...

chmod 777 /repo/SOLAS-Match/ui/templating/templates_compiled chmod 777 /repo/SOLAS-Match/ui/templating/cache

I have merged your changes into the develop branch (that is the branch pull requests should target) because the develop branch is what is used on the test server.

I will verify all is OK on dev server.

Alan.
aquilax commented 7 years ago

@alanbarrett Sorry, forgot to change the target branch. Apache needs write/create access for these because of Smarty, but certanly not 777, 775 should be sufficient if the owner:group are set correctly.