Development

#2719 ([PATCH] sfGuardPlugin Fatal Error on getIsSuperAdmin())

You must first sign up to be able to contribute.

Ticket #2719 (closed defect: fixed)

Opened 1 year ago

Last modified 6 months ago

[PATCH] sfGuardPlugin Fatal Error on getIsSuperAdmin()

Reported by: dcaravana Assigned to: francois
Priority: major Milestone:
Component: sfGuardPlugin Version: 1.0.10
Keywords: Cc: Kris.Wallsmith
Qualification: Design decision

Description

My app gives always this error (if you are not logged in):

Fatal error: Call to a member function getIsSuperAdmin() on a non-object in /home/xxx/webapps/editore/plugins/sfGuardPlugin/lib/user/sfGuardSecurityUser.class.php on line 39

On the web there's no fix, so I tried changing that line with:

return $this->getGuardUser() ? $this->getGuardUser()->getIsSuperAdmin() : false;

I'm not sure if this is the right fix because I do not know well symfony.

Attachments

sfGuardPlugin.patch (0.5 kB) - added by Pascal.Borreli on 12/25/07 10:06:59.
Patch for sfGuardSecurityUser.class.php

Change History

12/25/07 10:06:59 changed by Pascal.Borreli

  • attachment sfGuardPlugin.patch added.

Patch for sfGuardSecurityUser.class.php

12/25/07 10:16:18 changed by Pascal.Borreli

  • owner changed from fabien to Pascal.Borreli.
  • status changed from new to assigned.
  • qualification changed from Unreviewed to Design decision.
  • summary changed from sfGuardPlugin Fatal Error on getIsSuperAdmin() (includes solution) to [PATCH] sfGuardPlugin Fatal Error on getIsSuperAdmin().

I've included the patch to include your solution which looks good.

12/27/07 09:31:04 changed by Pascal.Borreli

  • owner changed from Pascal.Borreli to francois.
  • status changed from assigned to new.

Looks ok for me, waiting for core team's GO to include the patch.

03/06/08 01:45:24 changed by Kris.Wallsmith

  • cc set to Kris.Wallsmith.

I would like to see this implemented too.

05/26/08 20:52:10 changed by fabien

  • status changed from new to closed.
  • resolution set to fixed.

(In [9287]) sfGuardPlugin: fixed isSuperAdmin() method (closes #2719)