[Tickets #12133] Tasks in portal view may have an invalid share name

[noreply at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/12133
------------------------------------------------------------------------------
  Ticket             | 12133
  Created By         | lauffer at ph-freiburg.de
  Summary            | Tasks in portal view may have an invalid share name
  Queue              | Nag
  Version            | 4.0.2
  Type               | Bug
  State              | Unconfirmed
  Priority           | 1. Low
  Milestone          |
  Patch              | 1
  Owners             |
------------------------------------------------------------------------------


lauffer at ph-freiburg.de (2013-03-20 16:17) wrote:

Hi!

If you remove and add the "Tasks Summary" in the portal view  
without(!) saving the settings of the new Tasks Summary element you  
get an invalid share name (see protected function _getShares(array  
$ids) in Horde/Share/Sql.php).

It ends up in a error like:

   "Share name lauffer not found"

(funny: first we saw a invalid share name called "jan" at some user profiles).

The problem is caused by an invalid default value (afterwards stored  
in the horde_prefs at the show_tasklists key) with the login of the  
user. The function above expects a "crypted" share names but instead  
pass something matching for share owner.

You find the origin for this default  in our nag file lib/Block/Summary.php

I would say we could initial an empty array() instead, see the simple patch.

What would you say?

Best regards and thank you for your support,
Stephan



lauffer at ph-freiburg.de (2013-03-20 16:17) uploaded: nag_Summary.php.patch

http://bugs.horde.org/h/services/download/?module=whups&actionID=download_file&file=nag_Summary.php.patch&ticket=12133&fn=%2Fnag_Summary.php.patch