diff --git a/e107_plugins/newsletter/admin_config.php b/e107_plugins/newsletter/admin_config.php index 5890450ab..39aa33657 100644 --- a/e107_plugins/newsletter/admin_config.php +++ b/e107_plugins/newsletter/admin_config.php @@ -112,10 +112,6 @@ class newsletter $this->createIssue(); } - /*if($this -> message) - { - $ns->tablerender('', "
".$this -> message.'
'); - }*/ if($mes) { $ns->tablerender($caption, $mes->render() . $text); @@ -130,7 +126,7 @@ class newsletter $mes = e107::getMessage(); $tp = e107::getParser(); - if(!$sql->db_Select('newsletter', '*', "newsletter_parent='0' ORDER BY newsletter_id DESC")) + if(!$sql->select('newsletter', '*', "newsletter_parent='0' ORDER BY newsletter_id DESC")) { $mes->addInfo(NLLAN_05); } @@ -148,11 +144,11 @@ class newsletter ".LAN_ID." ".LAN_NAME." ".NLLAN_07." - ".LAN_OPTIONS." + ".LAN_OPTIONS." "; - $nlArray = $sql->db_getList(); + $nlArray = $sql->db_getList(); foreach($nlArray as $data) { $text .= " @@ -160,7 +156,11 @@ class newsletter ".$data['newsletter_id']." ".$data['newsletter_title']." ".((substr_count($data['newsletter_subscribers'], chr(1))!= 0)?"".substr_count($data['newsletter_subscribers'], chr(1))."":substr_count($data['newsletter_subscribers'], chr(1)))." - ".ADMIN_EDIT_ICON."toJS(LAN_CONFIRMDEL." [ID: ".$data['newsletter_id']." ]")."') \"/> + + ".ADMIN_EDIT_ICON." + toJS(LAN_CONFIRMDEL." [ID: ".$data['newsletter_id']." ]")."') \"/> + + "; } @@ -171,9 +171,9 @@ class newsletter } $ns->tablerender(NLLAN_10, $mes->render() . $text); - unset($text); // FIXME fix to prevent 'existing newsletters' table from showing twice, is this ok? + unset($text); - if(!$sql->db_Select('newsletter', '*', "newsletter_parent!='0' ORDER BY newsletter_id DESC")) + if(!$sql->select('newsletter', '*', "newsletter_parent!='0' ORDER BY newsletter_id DESC")) { $mes->addinfo(NLLAN_11); } @@ -193,7 +193,7 @@ class newsletter ".NLLAN_12." ".NLLAN_13." ".NLLAN_14." - ".LAN_OPTIONS." + ".LAN_OPTIONS." "; @@ -207,7 +207,7 @@ class newsletter ".$data['newsletter_issue']." [ ".$data['newsletter_parent']." ] ".$data['newsletter_title']." ".($data['newsletter_flag'] ? LAN_YES : "toJS(NLLAN_18)."') \" />")." - ".ADMIN_EDIT_ICON."toJS(NLLAN_19." [ID: ".$data['newsletter_id']." ]")."') \"/> + ".ADMIN_EDIT_ICON."toJS(NLLAN_19." [ID: ".$data['newsletter_id']." ]")."') \"/> @@ -241,9 +241,13 @@ class newsletter $text .= "
+ + + + - + @@ -332,6 +336,10 @@ class newsletter $text .= "
".LAN_TITLE."".$frm->text('newsletter_title', $newsletter_title, '200')."
".LAN_DESCRIPTION."
+ + + + - + - + +
".NLLAN_30." @@ -348,20 +356,20 @@ class newsletter
".NLLAN_31."".$frm->text('newsletter_title', $newsletter_title, '200')."
".NLLAN_32."".$frm->number('newsletter_', $newsletter_issue, '200')."
".NLLAN_33." -
".display_help('helpa')."
- ".($edit ? $frm->admin_button('createIssue', LAN_UPDATE, 'update')."\n" : $frm->admin_button('createIssue', LAN_CREATE, 'submit'))." + ".($edit ? $frm->admin_button('createIssue', LAN_UPDATE, 'update')."" : $frm->admin_button('createIssue', LAN_CREATE, 'submit'))."
@@ -418,14 +426,14 @@ class newsletter $issue = intval(str_replace('nlmailnow_', '', $issue)); // Get details of current newsletter issue - if(!$sql->db_Select('newsletter', '*', 'newsletter_id='.$issue)) + if(!$sql->select('newsletter', '*', 'newsletter_id='.$issue)) { return FALSE; } - $newsletterInfo = $sql->db_Fetch(MYSQL_ASSOC); + $newsletterInfo = $sql->fetch(MYSQL_ASSOC); // Get parent details - has header/footer and subscriber list - if(!$sql -> db_Select('newsletter', '*', "newsletter_id='".$newsletterInfo['newsletter_parent']."' ")) + if(!$sql->select('newsletter', '*', "newsletter_id='".$newsletterInfo['newsletter_parent']."' ")) { return FALSE; } @@ -523,9 +531,9 @@ class newsletter { $sql = e107::getDb(); - if($sql->db_Select("newsletter", "*", "newsletter_id='{$id}'")) + if($sql->select("newsletter", "*", "newsletter_id='{$id}'")) { - $foo = $sql->db_Fetch(); + $foo = $sql->fetch(); if(!$foo['newsletter_parent']) { $this -> defineNewsletter($foo); @@ -552,14 +560,14 @@ class newsletter if(strpos($tmp['key'], 'newsletter') === 0) { $id = intval(str_replace('newsletter_', '', $tmp['key'])); - $sql->db_Delete('newsletter', "newsletter_id='{$id}'"); + $sql->delete('newsletter', "newsletter_id='{$id}'"); //$this -> message = NLLAN_42; $mes->addSuccess(LAN_DELETED); } else { $id = intval(str_replace('issue_', '', $tmp['key'])); - $sql->db_Delete('newsletter', "newsletter_id='{$id}' "); + $sql->delete('newsletter', "newsletter_id='{$id}' "); //$this -> message = NLLAN_43; $mes->addSuccess(LAN_DELETED); } @@ -686,7 +694,6 @@ class newsletter "; if ($_nl_sanatized == 1) { - //$vs_text .= "
".NLLAN_66."
"; $mes->addInfo(NLLAN_66); } @@ -698,8 +705,8 @@ class newsletter function remove_subscribers($p_id, $p_key) { $sql = e107::getDb(); - $sql -> db_Select('newsletter', '*', 'newsletter_id='.intval($p_id)); - if($nl_row = $sql-> db_Fetch(MYSQL_ASSOC)) + $sql ->select('newsletter', '*', 'newsletter_id='.intval($p_id)); + if($nl_row = $sql->fetch(MYSQL_ASSOC)) { $subscribers_list = array_flip(explode(chr(1), $nl_row['newsletter_subscribers'])); unset($subscribers_list[$p_key]); diff --git a/e107_plugins/newsletter/languages/English_admin_newsletter.php b/e107_plugins/newsletter/languages/English_admin_newsletter.php index 7e1865b12..209d92c91 100644 --- a/e107_plugins/newsletter/languages/English_admin_newsletter.php +++ b/e107_plugins/newsletter/languages/English_admin_newsletter.php @@ -8,8 +8,6 @@ * * Plugin - Newsletter * - * $URL$ - * $Id$ */ //define('NLLAN_01', 'Newsletter');