6.x-1.x-dev

I agree that HEAD should be where patches go. Although it's a bit late a branch DRUPAL-6--1 for the next stable release(s) would IMO be allright

It's been awhile, but I will attempt it ;-)

The issues that caused me to have a good look at workspace.module code were I think the following:

#470646: Authenticated user workspace access when not allowed.
#124912: Permission for who gets access to My workspace

However when I got stuck in I found a number of other issues that were causing errors, especially on an install in a fresh install of Drupal, eg no nodes or comments yet.

1) I found that the function workspace_build_rows() was throwing a 'division by zero' error when attempting to parse a result with no rows. I added some code to protect against this WSOD.

This does not apply to workspace_list_files() as it does not use workspace_build_rows()
Perhaps it should for the sake of consistency and ease of maintenance in the future but that's another issue.

BTW, you will only see this error if display_errors is set in php.ini

2) maxcomments is never set, so I added code to allow the user to choose a pagination value if comments are enabled.

3) The following code:
line 272

  $max = isset($account->workspaces) ? $account->workspaces['default']['maxcomments'] : 50;

This will not work because $account->workspaces can return TRUE even though $account->workspaces['default']['maxcomments'] is not defined so the default of 50 is never applied. This I think is the reason there have been reports about no Pagination. see #376082: Pagination

The same applies to line 219 and line 387

A patch on today's 6.x-1.x-dev is attached, thank you Frank for attending to this.

I have just done a new install on a fresh drupal 6.13
I have found that the division by zero error no longer occurs so presumably the function pager_sql() has been updated to just return.
I am also getting the default of '50' showing when it should instead of the '0' I was getting some months ago so I have removed those fixes, no longer needed.

Here is a new patch, it corrects a theming issue where the function theme_workspace_list() wes being called directly instead of via the theme() function. I have tested this by copying function theme_workspace_list() to my theme's template.php, renamed of course and altering the output from 'Your workspace is currently empty.' to 'You have no content.' Flush the cache and the new phrase appears. Tested on acquia-marina theme.

It also adds a textfield to the configure form when comments are enabled so that should sort #376082: Pagination

hope this helps

Just a few more issues, I made a mistake in the form, had maxfilenames where it should have been maxcomments. Also it wasn't saving maxcomments, and the theme() bug is fixed in the attached patch. Also fixes auto-sort on 'Modified' which was not working. I have also attached a patched copy of workspace.module. This should not affect the translation files

Getting there!

They are on today's cvs ;-)
I'll check that list though

#344585: "Add new item" feature produces bad URLs for CCK content types
yes but needs testing, I don't have a cck setup to test it on ATM

#470646: Authenticated user workspace access when not allowed.
yes but needs testing. It looks to me that it is working but others need to look too.

#376082: Pagination
yes but in a different way, all three tables are put through pager_query()

#488216: Default sort order
is also in the v 4 patch. It worked in 'files' but not content or comments, I haven't got any comments in my devbox sites but it works in content. as the code change is the same in both I reckon it will work.

when i click on comments, i'm still getting the division by zero error even with the latest dev version...

warning: Division by zero in /home/content/12/5865312/html/corporate/includes/pager.inc on line 73.

forget my last comment, module conflict...had ajax module installed, which was the cause of division by zero error...
disabled its form handling plugins and things are back to normal...
thanks for the great module...
joseph