In module source code is used 'maxcomments' parameter, which is not initialized. User can set only maximum number of nodes and filenames.
Code:
$max = isset($account->workspaces) ? $account->workspaces['default']['maxcomments'] : 50;
Should be replaced by:
$max = isset($account->workspaces) ? $account->workspaces['default']['maxnodes'] : 50;
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | workspace.module.patch | 2.86 KB | calmforce |
| #1 | workspace_comment.patch | 1.09 KB | hutch |
Comments
Comment #1
hutch commentedAlternately the option to set the number of comments could be added to the configuration page
Here is an untested patch on HEAD
Comment #2
frank ralf commentedComment #4
calmforce commentedThe patch in comment #1 doesn't work because the 'maxcomments' field value has not been saved in workspace_configure_form_submit function. I found one more bug related to 'maxfilenames' setting: function workspace_list_files uses unidentified variable $user instead of $account.
I fixed both problems and the patch is attached.