Products

Solutions

Resources

Partners

Community

Blog

About

QA

Ideas Test

New Community Website

Ordinarily, you'd be at the right spot, but we've recently launched a brand new community website... For the community, by the community.

Yay... Take Me to the Community!

Welcome to the DNN Community Forums, your preferred source of online community support for all things related to DNN.
In order to participate you must be a registered DNNizen

HomeHomeDevelopment and...Development and...DNN Platform (o...DNN Platform (o...GetUsersByRoleName Includes Expired Users?GetUsersByRoleName Includes Expired Users?
Previous
 
Next
New Post
6/12/2013 3:01 PM
 

Hello,

I was getting some weird results on something here and I've found now that the RoleControler's "GetUsersByRoleName" seems to return users who are expired in that role.

Is that a bug or by design or am I missing something?

Thanks,

Mike

 
New Post
6/13/2013 3:09 PM
 
its difficult to say off the top of my head as I can see a use-case for either scenario - if you raise it as a bug, someone will look at all the usages of it and assess it, but typically to resolve something such as this we have to introduce a new parameter as we cant arbitrarily change results unless it's very obvious it was a bug e.g. perhaps the list needs expired users to bind to the grid. The standard pattern would be to add a version of the API call and add a parameter such as ReturnExpiredUsers - that defaults to true for the existing usage and we add a new API call that uses the false version i.e. GetNonExpiredUsersByRoleName

Buy the new Professional DNN7: Open Source .NET CMS Platform book Amazon US
 
New Post
6/14/2013 5:15 PM
 

Thanks Cathal.

Yeah - Sebastian pointed out elsewhere that it's probably because DNN uses it to get the list of all roles for the administrative list - which includes expired people. He suggested the same thing from you here...

I'll just submit it as a gemini request - I don't really care how you'd do it - either with a parameter or a separate procedure - but I do feel like there should be a core way to get the users currently in a role.

Hope all's well,

Mike

 
Previous
 
Next
HomeHomeDevelopment and...Development and...DNN Platform (o...DNN Platform (o...GetUsersByRoleName Includes Expired Users?GetUsersByRoleName Includes Expired Users?


These Forums are dedicated to discussion of DNN Platform and Evoq Solutions.

For the benefit of the community and to protect the integrity of the ecosystem, please observe the following posting guidelines:

  1. No Advertising. This includes promotion of commercial and non-commercial products or services which are not directly related to DNN.
  2. No vendor trolling / poaching. If someone posts about a vendor issue, allow the vendor or other customers to respond. Any post that looks like trolling / poaching will be removed.
  3. Discussion or promotion of DNN Platform product releases under a different brand name are strictly prohibited.
  4. No Flaming or Trolling.
  5. No Profanity, Racism, or Prejudice.
  6. Site Moderators have the final word on approving / removing a thread or post or comment.
  7. English language posting only, please.
What is Liquid Content?
Find Out
What is Liquid Content?
Find Out
What is Liquid Content?
Find Out