Re^2: Weird Scratch Pad Glitch (found)

The problem looks fairly easy to fix, but I'll leave that for others at this point.

Im totally flummoxed by this. The scratch pads are named "$user\'s scratchpad" so linking to a node by that name should take one to the scratchpad. So I don't understand what the problem is that needs fixing. Otherwise id be happy to take care of it.

---
demerphq

_{First they ignore you, then they laugh at you, then they fight you, then you win.

-- Gandhi}

Comment on Re^2: Weird Scratch Pad Glitch (found) Download Code

Replies are listed 'Best First'.
Re^3: Weird Scratch Pad Glitch (found) by tye (Sage) on Jun 04, 2004 at 15:21 UTC
Line 10 is: `$query->param('node', "$user\'s scratchpad");` [download] which changes a global which changes what page the user gets sent to not based on what page the user wanted to go to. Save and restore the value of this parameter so that after this code is run the CGI parameters are as the user requested. Perhaps a worse problem is that we have tons of never-used user pages and the way scratchpads were implemented means that any time anyone visits one of these pages a blank and useless scratchpad is created. It'd be wise to avoid creating a node for any scratchpad that is empty (but don't delete a scratchpad node just because it is empty). - tye	[reply] [d/l]
Re^4: Weird Scratch Pad Glitch (found) by demerphq (Chancellor) on Jun 05, 2004 at 13:31 UTC
Save and restore the value of this parameter so that after this code is run the CGI parameters are as the user requested. Done Perhaps a worse problem is that we have tons of never-used user pages and the way scratchpads were implemented means that any time anyone visits one of these pages a blank and useless scratchpad is created. It'd be wise to avoid creating a node for any scratchpad that is empty (but don't delete a scratchpad node just because it is empty). This is fixed, with an additional benefit that the scratchpad viewer now reuses the same code as the scratchpad edit\|display pages. But at the (temporary) cost that View links on homenodes will be displayed regardless of status. This can/will be fixed, but the unecessary creation was higher priority I thought. Now a scratchpad will only be created if someone actually tries to view it. If you link to a user who doesnt have a scratchpad the link resolves to the Scratchpad Viewer which if followed will cause the creation of the node. If one does exist then it resolves to it directly. The always show the link is because I didnt have time to determine the privacy regardless of the existance of the new style node and i didnt want to force the creation of a new node just by viewing a homenode. --- demerphq _{First they ignore you, then they laugh at you, then they fight you, then you win. -- Gandhi}	[reply] [d/l]