ListMailPRO Email Marketing Software Forums

ListMailPRO Email Marketing Software Forums => Development, Suggestions => Topic started by: chufford on August 16, 2006, 11:54:19 am

Title: Process inter-list dependancies during imports
Post by: chufford on August 16, 2006, 11:54:19 am

I found something I consider a bug but it could be by design - what do  you guys think?

I have a list (let's call it "Registered List") that is set to automatically remove a user from a different list ("Demo List") when they subscribe. This is using the "When users are added to this list remove them from the following list(s):" feature. If I do a batch import of users to the registered list, I would expect it to also remove the users from the "Demo" list. However, if the user is already on the Registered list, it is seen as a duplicate and is not added but it then does not check to see if it should be removed from the Demo list.

So for me, I'd like the inter-list dependancies tested whether or not the email address is a duplicate - both for subscribes and for imports.

Is my description clear? Does that make sense? What do you think?

Thanks,
Chris

Title: Process inter-list dependancies during imports
Post by: chufford on August 27, 2006, 08:32:22 am

So it looks like this issue hasn't come up for anyone else. Dean, what do you think? Can you direct me to the location in the code where the import/inter-list dependancies occurs? I'd like to look into reversing the order those actions are performed.

Thanks.

Title: Process inter-list dependancies during imports
Post by: DW on August 27, 2006, 06:02:09 pm

In edituser.php...

Try adding this:

Code: [Select]

if($listopts[2]=='1') remlists($email,$lis,2);
if($listopts[3]=='1') addlists($email,$lis,3);
After the line:

Code: [Select]

$isadupe = 1;
I'll have this in the next update (though, it may not be implemented the same:))

Let me know how you fare - I didn't test this.

Regards

Title: Process inter-list dependancies during imports
Post by: chufford on August 29, 2006, 11:30:39 pm

That did what I wanted. Thanks.