A value of -1 would be that returned for the anonymous user. I see nothing wrong with your code so have to ask if . . .

1. When testing our code, have you first logged in as a valid user?

2. You are calling this method from within a module control which inherits from PortalModuleBase?

3. Assuming that #2 is YES, have you tried using the provided properties UserInfo and UserId. I would suspect that UserId would also return -1 since the property also relies on the same method, GetCurrentUserInfo, that you are calling for its data.

---

Bill, WESNet Designs
Team Lead - DotNetNuke Gallery Module Project (Not Actively Being Developed)
Extensions Forge Projects . . .
Current: UserExport, ContentDeJour, ePrayer, DNN NewsTicker, By Invitation
Coming Soon: FRBO-For Rent By Owner

If the .aspx page loaded in the IFrame module is located within your DNN website domain, the call to GetCurrentUserInfo should work just fine. If, however, the .aspx page is located outsite of the DNN website's domain, it's HttpContext will not contain a UserInfo object causing the GetCurrentUserInfo to fail and return a new UserInfo object which will have a UserId of -1. If that is your case, you will need to configure the IFrame module to pass the DNN UserId, etc. to your .aspx page using the query string.

---

Bill, WESNet Designs
Team Lead - DotNetNuke Gallery Module Project (Not Actively Being Developed)
Extensions Forge Projects . . .
Current: UserExport, ContentDeJour, ePrayer, DNN NewsTicker, By Invitation
Coming Soon: FRBO-For Rent By Owner