{"action":"created","issue":{"url":"https://api.github.com/repos/owncloud/core/issues/19187","labels_url":"https://api.github.com/repos/owncloud/core/issues/19187/labels{/name}","comments_url":"https://api.github.com/repos/owncloud/core/issues/19187/comments","events_url":"https://api.github.com/repos/owncloud/core/issues/19187/events","html_url":"https://github.com/owncloud/core/issues/19187","id":107328742,"number":19187,"title":"External storage deleted and no longer works after 8.1.3 upgrade","user":{"login":"keynet","id":5302023,"avatar_url":"https://avatars.githubusercontent.com/u/5302023?v=3","gravatar_id":"","url":"https://api.github.com/users/keynet","html_url":"https://github.com/keynet","followers_url":"https://api.github.com/users/keynet/followers","following_url":"https://api.github.com/users/keynet/following{/other_user}","gists_url":"https://api.github.com/users/keynet/gists{/gist_id}","starred_url":"https://api.github.com/users/keynet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/keynet/subscriptions","organizations_url":"https://api.github.com/users/keynet/orgs","repos_url":"https://api.github.com/users/keynet/repos","events_url":"https://api.github.com/users/keynet/events{/privacy}","received_events_url":"https://api.github.com/users/keynet/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/owncloud/core/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/owncloud/core/labels/needs%20info","name":"needs info","color":"0b02e1"}],"state":"closed","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/owncloud/core/milestones/37","html_url":"https://github.com/owncloud/core/milestones/8.2-current","labels_url":"https://api.github.com/repos/owncloud/core/milestones/37/labels","id":993629,"number":37,"title":"8.2-current","description":"","creator":{"login":"craigpg","id":160667,"avatar_url":"https://avatars.githubusercontent.com/u/160667?v=3","gravatar_id":"","url":"https://api.github.com/users/craigpg","html_url":"https://github.com/craigpg","followers_url":"https://api.github.com/users/craigpg/followers","following_url":"https://api.github.com/users/craigpg/following{/other_user}","gists_url":"https://api.github.com/users/craigpg/gists{/gist_id}","starred_url":"https://api.github.com/users/craigpg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/craigpg/subscriptions","organizations_url":"https://api.github.com/users/craigpg/orgs","repos_url":"https://api.github.com/users/craigpg/repos","events_url":"https://api.github.com/users/craigpg/events{/privacy}","received_events_url":"https://api.github.com/users/craigpg/received_events","type":"User","site_admin":false},"open_issues":223,"closed_issues":923,"state":"open","created_at":"2015-02-24T16:19:37Z","updated_at":"2015-09-24T10:23:58Z","due_on":"2015-10-20T04:00:00Z","closed_at":null},"comments":16,"created_at":"2015-09-19T12:00:13Z","updated_at":"2015-09-24T10:28:59Z","closed_at":"2015-09-19T14:32:01Z","body":"<h4>Server:</h4>\r\nSLES 11 SP4 / PHP 5.6.9\r\n\r\n<h4>Steps taken:</h4>\r\nUpgrade failed via repository route due to issues mentioned elsewhere (packages being recombined)\r\nInstalled fresh copy of 8.1.3  via tarball, set permissions on files to Apache's\r\nAutomatic database upgrade worked OK\r\nHowever external storage (local files) deleted. \r\nWhen re-adding, green light comes on momentarily, then goes red.\r\nLog excerpt below for any directory attempted, including ones with full permissions\r\nBug, problem with the database, or other?\r\n```\r\nSep 19 12:40:25 ha-server ownCloud[11123]: {PHP} Undefined index: owner at /srv/www/htdocs/owncloud/apps/files/index.php#140\r\nSep 19 12:40:25 ha-server ownCloud[11123]: {PHP} Undefined index: ownerDisplayName at /srv/www/htdocs/owncloud/apps/files/index.php#141\r\nSep 19 12:40:25 ha-server ownCloud[11123]: {PHP} Undefined index: owner at /srv/www/htdocs/owncloud/apps/files/lib/helper.php#56\r\nSep 19 12:40:25 ha-server ownCloud[11123]: {PHP} Undefined index: ownerDisplayName at /srv/www/htdocs/owncloud/apps/files/lib/helper.php#57\r\nSep 19 12:40:28 ha-server ownCloud[11476]: {PHP} Array to string conversion at /srv/www/htdocs/owncloud/lib/private/template/functions.php#36\r\nSep 19 12:40:48 ha-server ownCloud[11587]: {PHP} file_put_contents(data/mount.json): failed to open stream: No such file or directory at /srv/www/htdocs/owncloud/apps/files_external/lib/config.php#711\r\nSep 19 12:40:48 ha-server ownCloud[11587]: {PHP} chmod(): No such file or directory at /srv/www/htdocs/owncloud/apps/files_external/lib/config.php#712\r\n```\r\nThanks"},"comment":{"url":"https://api.github.com/repos/owncloud/core/issues/comments/142885336","html_url":"https://github.com/owncloud/core/issues/19187#issuecomment-142885336","issue_url":"https://api.github.com/repos/owncloud/core/issues/19187","id":142885336,"user":{"login":"keynet","id":5302023,"avatar_url":"https://avatars.githubusercontent.com/u/5302023?v=3","gravatar_id":"","url":"https://api.github.com/users/keynet","html_url":"https://github.com/keynet","followers_url":"https://api.github.com/users/keynet/followers","following_url":"https://api.github.com/users/keynet/following{/other_user}","gists_url":"https://api.github.com/users/keynet/gists{/gist_id}","starred_url":"https://api.github.com/users/keynet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/keynet/subscriptions","organizations_url":"https://api.github.com/users/keynet/orgs","repos_url":"https://api.github.com/users/keynet/repos","events_url":"https://api.github.com/users/keynet/events{/privacy}","received_events_url":"https://api.github.com/users/keynet/received_events","type":"User","site_admin":false},"created_at":"2015-09-24T10:28:59Z","updated_at":"2015-09-24T10:28:59Z","body":"I see you're having a good debate! And I don't think the users are quite as stupid as some suggest - they just don't know the code (my point above) like the devs, and can't be expected to.\r\n\r\nI think the approach can be improved  - IMHO the config file is trying to do too many things:\r\na) Holding parameters that are user configurable and not changed by OC\r\ne.g 'mail_domain' => 'example.com',\r\nb) Holding parameters that are system dependent and/or version dependent\r\ne.g  'version' => '7.0.2.1'\r\nc) Holding parameters that can get (over)written by OC while running or at update time\r\ne.g.  'mail_from_address' => 'owncloud',\r\n\r\nTrying to do so many things in 1 file is too difficult IMHO.\r\nMy suggestion is to have separate configs for the separate use cases. Also improves security as the owner can be set to root or at least read only for the fixed params.\r\n\r\nI also think the documentation has too many (yes too many) words. I know someone is trying to explain, but it would IMHO be better to have a table that lists, for each parameter:\r\nName, description, default setting, updated by OC (when)?, Notes (big fat warnings etc)\r\n\r\nMy example is mkvtoolnix, but even he could add more columns as I suggest above to help clarity\r\nhttps://www.bunkus.org/videotools/mkvtoolnix/doc/mkvmerge.html\r\n\r\nHTH"}}