{"action":"created","issue":{"url":"https://api.github.com/repos/splitbrain/dokuwiki/issues/1592","repository_url":"https://api.github.com/repos/splitbrain/dokuwiki","labels_url":"https://api.github.com/repos/splitbrain/dokuwiki/issues/1592/labels{/name}","comments_url":"https://api.github.com/repos/splitbrain/dokuwiki/issues/1592/comments","events_url":"https://api.github.com/repos/splitbrain/dokuwiki/issues/1592/events","html_url":"https://github.com/splitbrain/dokuwiki/pull/1592","id":156775779,"number":1592,"title":"Fix bulk upload to allow multiple groups","user":{"login":"ltning","id":1304442,"avatar_url":"https://avatars.githubusercontent.com/u/1304442?v=3","gravatar_id":"","url":"https://api.github.com/users/ltning","html_url":"https://github.com/ltning","followers_url":"https://api.github.com/users/ltning/followers","following_url":"https://api.github.com/users/ltning/following{/other_user}","gists_url":"https://api.github.com/users/ltning/gists{/gist_id}","starred_url":"https://api.github.com/users/ltning/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ltning/subscriptions","organizations_url":"https://api.github.com/users/ltning/orgs","repos_url":"https://api.github.com/users/ltning/repos","events_url":"https://api.github.com/users/ltning/events{/privacy}","received_events_url":"https://api.github.com/users/ltning/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-05-25T15:10:44Z","updated_at":"2016-05-25T17:15:33Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/splitbrain/dokuwiki/pulls/1592","html_url":"https://github.com/splitbrain/dokuwiki/pull/1592","diff_url":"https://github.com/splitbrain/dokuwiki/pull/1592.diff","patch_url":"https://github.com/splitbrain/dokuwiki/pull/1592.patch"},"body":"When bulk importing users, only the first group listed is assigned. This is due to the direct addressing of the candidate array, and groups being separated by commas. My change slices off the fourth and any following array pieces and joins them by commas.\r\n\r\nTested on Release 2015-08-10a \"Detritus\"."},"comment":{"url":"https://api.github.com/repos/splitbrain/dokuwiki/issues/comments/221643796","html_url":"https://github.com/splitbrain/dokuwiki/pull/1592#issuecomment-221643796","issue_url":"https://api.github.com/repos/splitbrain/dokuwiki/issues/1592","id":221643796,"user":{"login":"Chris--S","id":176843,"avatar_url":"https://avatars.githubusercontent.com/u/176843?v=3","gravatar_id":"","url":"https://api.github.com/users/Chris--S","html_url":"https://github.com/Chris--S","followers_url":"https://api.github.com/users/Chris--S/followers","following_url":"https://api.github.com/users/Chris--S/following{/other_user}","gists_url":"https://api.github.com/users/Chris--S/gists{/gist_id}","starred_url":"https://api.github.com/users/Chris--S/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Chris--S/subscriptions","organizations_url":"https://api.github.com/users/Chris--S/orgs","repos_url":"https://api.github.com/users/Chris--S/repos","events_url":"https://api.github.com/users/Chris--S/events{/privacy}","received_events_url":"https://api.github.com/users/Chris--S/received_events","type":"User","site_admin":false},"created_at":"2016-05-25T17:15:33Z","updated_at":"2016-05-25T17:15:33Z","body":"I'm concerned that we shouldn't try and fix problems elsewhere - e.g. buggy str_getcsv, especially if we don't understand the problem.\r\n\r\nI've run a few simple tests both on str_getcsv in isolation and dokuwiki's user import function and not had a problem.  I'm running PHP 5.5.31.\r\n\r\nI've also had a search in PHP bug tracker.  There is one recent bug report for php 5.6.13, but I don't understand the report, there is no sample data so I can't tell if its relevant or not (https://bugs.php.net/bug.php?id=70491)\r\n\r\nCan you post an actual problem line?\r\nWhat version of PHP are you running?"}}
{"actor":{"display_login":"Chris--S"}}