{"issue":{"pull_request":{"diff_url":"https://github.com/kbond/sfAdminDashPlugin/pull/6.diff","html_url":"https://github.com/kbond/sfAdminDashPlugin/pull/6","patch_url":"https://github.com/kbond/sfAdminDashPlugin/pull/6.patch"},"state":"open","user":{"avatar_url":"https://secure.gravatar.com/avatar/0a198e694d5eac4ae8f6a24ddb92e2c4?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"0a198e694d5eac4ae8f6a24ddb92e2c4","url":"https://api.github.com/users/kbsali","login":"kbsali","id":53676},"milestone":null,"updated_at":"2011-11-08T19:27:50Z","labels":[],"assignee":null,"comments":2,"closed_at":null,"url":"https://api.github.com/repos/kbond/sfAdminDashPlugin/issues/6","body":"This simply remove the dependency the plugin has with sfGuardUser plugin : the dashboard and the global drop down menu won't list any items if $sf_user->isAuthenticated() does not return true.\r\nNow setting the option is_protected to false this check is skipped and the menus work fine.","html_url":"https://github.com/kbond/sfAdminDashPlugin/issues/6","title":"No sf guard dependency","number":6,"id":2176786,"created_at":"2011-11-08T17:18:02Z"},"action":"created","comment":{"user":{"avatar_url":"https://secure.gravatar.com/avatar/1d62d8c50932c89c26bb8a919d9022e5?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"1d62d8c50932c89c26bb8a919d9022e5","url":"https://api.github.com/users/CraftyShadow","login":"CraftyShadow","id":202527},"updated_at":"2011-11-08T19:27:50Z","url":"https://api.github.com/repos/kbond/sfAdminDashPlugin/issues/comments/2672263","body":"This is not a dependency on sfGuardPlugin. The method isAuthenticated() is a part of sfBasicSecurityUser:\r\nhttps://github.com/symfony/symfony1/blob/1.4/lib/user/sfBasicSecurityUser.class.php#L177\r\n\r\nI am not sure how I feel about this change. It's basically a question of whether there is a use case when you will want to have user authentication active, and at the same time have it not matter for the sfAdminDash menu.\r\n\r\nIs your use case like that?","id":2672263,"created_at":"2011-11-08T19:27:50Z"}}