summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChristian Fowler <spider@viovio.com>2010-02-26 00:59:41 +0000
committerChristian Fowler <spider@viovio.com>2010-02-26 00:59:41 +0000
commitf0bd3b1323c6685e201edfe6fecbb8a3db3be38d (patch)
tree3d222f2f85617938642e10bb752b11c6291d788c
parenta4f6cf468dea2bbe2813c0fdef392788a01cb0ec (diff)
downloadnewsletters-f0bd3b1323c6685e201edfe6fecbb8a3db3be38d.tar.gz
newsletters-f0bd3b1323c6685e201edfe6fecbb8a3db3be38d.tar.bz2
newsletters-f0bd3b1323c6685e201edfe6fecbb8a3db3be38d.zip
add newsletter optin hidden field so mass unsubscribes don't happen
-rw-r--r--bit_setup_inc.php9
-rw-r--r--templates/user_register_inc.tpl1
2 files changed, 8 insertions, 2 deletions
diff --git a/bit_setup_inc.php b/bit_setup_inc.php
index bb20d28..3bad4fc 100644
--- a/bit_setup_inc.php
+++ b/bit_setup_inc.php
@@ -38,19 +38,24 @@ if( $gBitSystem->isPackageActive( NEWSLETTERS_PKG_NAME ) ) {
}
function newsletters_user_register( &$pObject ) {
+ if( !empty( $_REQUEST['newsletter_optin'] ) ) {
+ // hidden flag to indicate at least one newsletter was displayed
require_once NEWSLETTERS_PKG_PATH.'BitNewsletter.php';
$newsletter = new BitNewsletter();
$pParamHash = array();
$newsletters = $newsletter->getList($pParamHash);
foreach ($newsletters as $nl){
- if(!empty($_REQUEST['subscribe'])){//we want to stay in at least one, which requires more complicated checking
+ if( !empty( $_REQUEST['subscribe'] ) ){
+ // we want to stay in at least one, which requires more complicated checking
if( !in_array($nl['nl_id'], $_REQUEST['subscribe'])){ //not checked, implies unsubscribe
$newsletter->unsubscribe($nl['nl_id'],false);
}
- }else{ //we wish to unsubscribe from all newsletters
+ } else {
+ //we wish to unsubscribe from all newsletters
$newsletter->unsubscribe($nl['nl_id'],false);
}
}
+ }
}
}
?>
diff --git a/templates/user_register_inc.tpl b/templates/user_register_inc.tpl
index bd1879c..a2632fe 100644
--- a/templates/user_register_inc.tpl
+++ b/templates/user_register_inc.tpl
@@ -1,4 +1,5 @@
{if $newsletters}
+<input type="hidden" name="newsletter_optin" value="y"/>
<script type = "text/javascript">
{literal}
function unsubscribe_all(){