In regard to DRY, If the contents of the function were less trivial, I'd agree with you.  Like for the `getAllowEdit()` it makes sense.  But for this method in particular you're wrapping a trivial function inside of another: `Mage::getSingleton('admin/session')`.

So even if this were used in six different places, you could call `Mage::getSingleton('admin/session')` in all of those places and you'd have no more code duplication than if you called the `getAdminSession()` helper method?

But I get what you're saying about component isolation - good call!  :thumbsup: