Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

different default values are used for parameter 'asset-pipeline.enabled` #48

Closed
j-ed opened this issue Jan 16, 2017 · 1 comment
Closed
Labels

Comments

@j-ed
Copy link

j-ed commented Jan 16, 2017

I was asked by @nickvergessen to reopen this ticket in this repository again, because the referred parameter asset-pipeline.enabled doesn't exist anymore.

refers to nextcloud/server#3108
refers to nextcloud/server#3069

Expected behavior

For all parameters default values should be set and the same default value should be used for all appearances in the source code.

Current behavior

Not all of the parameters are set to a fixed default value, nor have they the same default value assigned in the source code.

As far as I understood the default value of the parameter asset-pipeline.enabled should be false but other values have been assigned in the source code:

getSystemValue('asset-pipeline.enabled', false)     # ok, default value
- /var/www/htdocs/nextcloud/apps2/calendar/controller/viewcontroller.php:108
- /var/www/htdocs/nextcloud/apps2/calendar/controller/viewcontroller.php:169

+ getSystemValue('asset-pipeline.enabled')          # <<< missing value, should be 'false'
  - /var/www/htdocs/nextcloud/apps/survey_client/lib/Categories/Server.php:74

Steps to reproduce

  1. search and compare the assigned values in the getSystemValue() call.

Environment

Server Configuration

OS: Linux 3.2.82
Web server: Apache2 2.4.25
Database: MariaDB 5.5.53
PHP version: 5.6.29
Nextcloud version: 11.0.0

Client Configuration

Browser: Mozilla Firefox 50.1.0
Operating system: Windows 7

@nickvergessen
Copy link
Member

The value is of no use at all nowerdays

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants