You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@allura.apache.org by Dave Brondsema <br...@users.sf.net> on 2014/04/14 23:41:49 UTC

[allura:tickets] #7326 API cleanup: rename `app.has_access` to indicate it is mail-specific



---

** [tickets:#7326] API cleanup: rename `app.has_access` to indicate it is mail-specific **

**Status:** open
**Milestone:** limbo
**Labels:** cleanup allura-api 
**Created:** Mon Apr 14, 2014 09:41 PM UTC by Dave Brondsema
**Last Updated:** Mon Apr 14, 2014 09:41 PM UTC
**Owner:** nobody

Almost all `has_access` usage is from helpers and security packages.  The `app.has_access` method is used for mail tasks:

`Allura/allura/tasks/mail_tasks.py:59:                    if not app.has_access(c.user, userpart):`

And documented in the first paragraph of https://forge-allura.apache.org/docs/platform_tour.html#email-integration

I think it should be named something different (and fall back to `has_access` for backwards compatibility)


---

Sent from sourceforge.net because dev@allura.apache.org is subscribed to https://sourceforge.net/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/allura/admin/tickets/options.  Or, if this is a mailing list, you can unsubscribe from the mailing list.