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

HomeHomeDNN Open Source...DNN Open Source...Module ForumsModule ForumsStoreStoreBug report - version 2.01.09Bug report - version 2.01.09
Previous
 
Next
New Post
12/18/2008 1:53 PM
 

We can think of a workaround to hide this error, but there is a real issue here anyway.

How to handle unsubscribed customers when they still have an id in the store tables (either with commands pending, or former customers) ?

Settings integrity constraints is a problem, because it would impact the core tables as well, something that we want to avoid. Anyway I cannot imagine letting a user deleting our store logs in cascade just because he decide to unsubscribe from the website. On the other hand we cannot prohibit unsubscribing just because you have been a store customer once.

We could add a customer table. Yet another user table in DNN... not very elegant solution for an 'integrated web platform'

We could just replicate the customer's name, so we could keep this information after a deletion from DNN' users tables.

We can also suggest the core team to add the concept of virtual deletion in the users tables - a 'isdeleted' field wich would not physically delete the user if he has been a store customer. Not easy I guess.

I you can see, there is no easy way to handle unsubcribed users, but we might at least avoid an ugly error message.

   

 

 


Benoît Sarton
www.bsi.fr
www.dotnetnuke.fr
 
New Post
12/20/2008 7:15 AM
 

Hi Alex,
thanks for navigation, I found the link button finally. I hadn’t reported to Gemini yet because there is possibility to select only Store module version 2.01.00 which is current release. This buggy version is pre-release of the next store module version and this code is new in this pre-release version. Also I think somebody from the store module team should take this report into consideration and correct the code before the next release.
 

 
New Post
12/20/2008 7:18 AM
 

Hi Benoit,
I agree with your thoughts. These are all legitimate questions about overall architecture. There are many other possibilities too, I am sure. But for now we should ensure application stability as the highest priority. Fortunately this is very easy to achieve. Just follow my two suggestions:

  1. Don’t use null object in function GetRegistrationAddress()
  2. Catch ArgumentOutOfRangeException exception in Page_Load event of CustomerAdmin control.

As result the customer order displays without address information. Now when our application doesn’t crash we can discuss about better integration of store module with dotnetnuke core framework.
 

 
New Post
12/21/2008 1:34 PM
 

Hi Jan,

Thanks for your feedback ! ;-)

This is corrected now. I followed your suggestion about catching ArgumentOutOfRangeException and added an error message inside getShipToAddress and getBillingToAddress (CustomerOrders.ascx.cs) where GetRegistrationAddress() is called. If a user account is deleted, the ShipTo and BillingTo Addresses will show "This user has been deleted (ID=n). Address is no longer available.". This will allow to find the user addres if a database backup is available.

Gilles


We (team members) are Humans offering their knowledge, their work and their spare time FOR FREE to benefit the community. It would be so particularly appreciated that your messages begin with "Hello" and end with "Thank you" or any other form of politeness. Ask yourself what your reaction would be, if you were approached by me (a total stranger) on the street to ask you something without saying "Hello" nor "Thank you"? After several years of services dedicated to the community, I begin to be tired to read requests without any form of politeness.
 
New Post
12/22/2008 10:29 AM
 

Hi Gilles,
nice hear from you again. I haven’t had time yet to explore dnn framework core code but maybe this problem can be solved more elegant by some art of deleted user archive. If store module can subscribe to by core framework published event in conjunction with user deletion then it can archive relevant user information to store specific table (for example). By the way, currently, portal admin receives all needed info by email when user unsubscribe from portal.
 

 
Previous
 
Next
HomeHomeDNN Open Source...DNN Open Source...Module ForumsModule ForumsStoreStoreBug report - version 2.01.09Bug report - version 2.01.09


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