Additional Admin Switches chmod /pub and Categories/Products Go To

Additional switches needed.

Configuring default setting for Categories/Products Go To for sites with Large categories
Configuration chmod setting for /pub for sites that cannot use hard coded 777
<a href='#crh-start'></a><a href='#crh-data-%7B%22processed%22%3A%20%5B%22https%3A//github.com/zencart/zc-v1-series/pull/148%23discussion_r17261767%22%2C%20%22https%3A//github.com/zencart/zc-v1-series/pull/148%23discussion_r17396708%22%2C%20%22https%3A//github.com/zencart/zc-v1-series/pull/148%23discussion_r17435184%22%5D%2C%20%22comments%22%3A%20%7B%22Pull%20bb4ea7ab9c78a918dde5835be51247e2c7c42a19%20zc_install/sql/install/mysql_zencart.sql%2012%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/zencart/zc-v1-series/pull/148%23discussion_r17261767%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22Should%20this%20be%20left%20as%20a%20free-form%20input%20field%3F%20Or%20limit%20to%20a%20few%20acceptable%20values%2C%20like%20777%2C%20755%2C%20655%2C%20644%20and%20merely%20suggest%20that%20755%20be%20the%20usual%20choice%3F%22%2C%20%22created_at%22%3A%20%222014-09-08T19%3A45%3A01Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/404472%3Fv%3D2%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/drbyte%22%7D%7D%2C%20%7B%22body%22%3A%20%22That%27s%20business%20logic%20that%20is%20best%20handled%20by%20the%20application%2C%20not%20the%20database.%22%2C%20%22created_at%22%3A%20%222014-09-10T23%3A48%3A43Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/1590605%3Fv%3D2%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/texdc%22%7D%7D%2C%20%7B%22body%22%3A%20%22I%20think%20what%20%40drbyte%20was%20suggesting%20is%20that%20the%20configuration%20settings%20can%20define%20a%20enum/select%20rather%20than%20a%20free%20form%2C%20and%20doing%20so%20would%20mitigate%20against%20users%20entering%20some%20%27stupid%27%20value.%20%22%2C%20%22created_at%22%3A%20%222014-09-11T17%3A35%3A13Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/227354%3Fv%3D2%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/zcwilt%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20zc_install/sql/install/mysql_zencart.sql%3AL2672-2679%22%7D%7D%7D'></a>
<a href='https://www.codereviewhub.com/'><img src='http://www.codereviewhub.com/site/github-bar.png' height=40></a>
- [ ] <a href='#crh-comment-Pull bb4ea7ab9c78a918dde5835be51247e2c7c42a19 zc_install/sql/install/mysql_zencart.sql 12'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/zencart/zc-v1-series/pull/148#discussion_r17261767'>File: zc_install/sql/install/mysql_zencart.sql:L2672-2679</a></b>
- <a href='https://github.com/drbyte'><img border=0 src='https://avatars.githubusercontent.com/u/404472?v=2' height=16 width=16'></a> Should this be left as a free-form input field? Or limit to a few acceptable values, like 777, 755, 655, 644 and merely suggest that 755 be the usual choice?
- <a href='https://github.com/texdc'><img border=0 src='https://avatars.githubusercontent.com/u/1590605?v=2' height=16 width=16'></a> That's business logic that is best handled by the application, not the database.
- <a href='https://github.com/zcwilt'><img border=0 src='https://avatars.githubusercontent.com/u/227354?v=2' height=16 width=16'></a> I think what @drbyte was suggesting is that the configuration settings can define a enum/select rather than a free form, and doing so would mitigate against users entering some 'stupid' value.


<a href='https://www.codereviewhub.com/zencart/zc-v1-series/pull/148?mark_as_completed=1'><img src='http://www.codereviewhub.com/site/github-mark-as-completed.png' height=26></a>&nbsp;<a href='https://www.codereviewhub.com/zencart/zc-v1-series/pull/148?approve=1'><img src='http://www.codereviewhub.com/site/github-approve.png' height=26></a>&nbsp;<a href='https://github.com/zencart/zc-v1-series/pull/148'><img src='http://www.codereviewhub.com/site/github-refresh.png' height=26></a>
<a href='#crh-end'></a>
I'm aware of that and it was the motivation for my comment:  business logic, such as restricting a set of valid values, belongs in the app not the db.

> On Sep 11, 2014, at 12:35 PM, Ian Wilson <notifications@github.com> wrote:
> 
> In zc_install/sql/install/mysql_zencart.sql:
> 
> > @@ -2671,6 +2672,7 @@ INSERT INTO configuration (configuration_title, configuration_key, configuration
> >  INSERT INTO configuration (configuration_title, configuration_key, configuration_value, configuration_description, configuration_group_id, sort_order, set_function, date_added) VALUES ('Enable Downloads', 'DOWNLOAD_ENABLED', 'true', 'Enable the products download functions.', '13', '1', 'zen_cfg_select_option(array(\'true\', \'false\'), ', now());
> >  INSERT INTO configuration (configuration_title, configuration_key, configuration_value, configuration_description, configuration_group_id, sort_order, set_function, date_added) VALUES ('Download by Redirect', 'DOWNLOAD_BY_REDIRECT', 'true', 'Use browser redirection for download. Disable on non-Unix systems.<br /><br />Note: Set /pub to 777 when redirect is true', '13', '2', 'zen_cfg_select_option(array(\'true\', \'false\'), ', now());
> >  INSERT INTO configuration (configuration_title, configuration_key, configuration_value, configuration_description, configuration_group_id, sort_order, set_function, date_added) VALUES ('Download by streaming', 'DOWNLOAD_IN_CHUNKS', 'false', 'If download-by-redirect is disabled, and your PHP memory_limit setting is under 8 MB, you might need to enable this setting so that files are streamed in smaller segments to the browser.<br /><br />Has no effect if Download By Redirect is enabled.', '13', '2', 'zen_cfg_select_option(array(\'true\', \'false\'), ', now());
> > +INSERT INTO configuration (configuration_title, configuration_key, configuration_value, configuration_description, configuration_group_id, sort_order, set_function, date_added) VALUES ('Set Download directory chmod setting' ,'DOWNLOAD_CHMOD', '777', 'Set Download directory chmod setting to 777 or 755.', '13', '3', '', now());
> I think what @drbyte was suggesting is that the configuration settings can define a enum/select rather than a free form, and doing so would mitigate against users entering some 'stupid' value.
> 
> —
> Reply to this email directly or view it on GitHub.