mirror/php-e107
1
0
Fork 0
mirror of https://github.com/e107inc/e107.git synced 2025-08-06 14:46:56 +02:00
Code Issues Releases Wiki Activity

e_form 'selected' bug, newspost various markup fixes

Browse Source
This commit is contained in:
secretr
2009-05-10 17:31:51 +00:00
parent 8df9b45984
commit 6a70db6ca4
2 changed files with 33 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

39
e107_admin/newspost.php
View File

@@ -9,9 +9,9 @@
* News Administration * News Administration
* *
* $Source: /cvs_backup/e107_0.8/e107_admin/newspost.php,v $ * $Source: /cvs_backup/e107_0.8/e107_admin/newspost.php,v $
* $Revision: 1.37 $ * $Revision: 1.38 $
* $Date: 2009-05-08 21:50:19 $ * $Date: 2009-05-10 17:31:50 $
* $Author: e107steved $ * $Author: secretr $
*/ */
require_once("../class2.php"); require_once("../class2.php");
@@ -985,10 +985,8 @@ class admin_newspost
<tr> <tr>
<td class='label'>".NWSLAN_15.":</td> <td class='label'>".NWSLAN_15.":</td>
<td class='control'> <td class='control'>
". ".$frm->radio('news_allow_comments', 0, $_POST['news_allow_comments'])."".$frm->label(LAN_ENABLED, 'news_allow_comments', 0)."&nbsp;&nbsp;
$frm->radio('news_allow_comments', 0, $_POST['news_allow_comments'])."".$frm->label(LAN_ENABLED, 'news_allow_comments', 1)."&nbsp;&nbsp;". ".$frm->radio('news_allow_comments', 1, $_POST['news_allow_comments'])."".$frm->label(LAN_DISABLED, 'news_allow_comments', 1)."
$frm->radio('news_allow_comments', 1, $_POST['news_allow_comments'])."".$frm->label(LAN_DISABLED, 'news_allow_comments', 1)
."
<div class='field-help'> <div class='field-help'>
".NWSLAN_18." ".NWSLAN_18."
</div> </div>
@@ -1644,14 +1642,25 @@ class admin_newspost
$text = " $text = "
<form method='post' action='".e_SELF."?maint' id='core-newspost-maintenance-form'> <form method='post' action='".e_SELF."?maint' id='core-newspost-maintenance-form'>
<div style='text-align:center'> <fieldset id='core-newspost-maintenance'>
<table class='fborder' style='".ADMIN_WIDTH."'> <legend class='e-hideme'>".LAN_NEWS_59."</legend>
<table class='adminform' cellpadding='0' cellspacing='0'>
<tr><td class='forumheader3'>".LAN_NEWS_56."</td><td style='text-align:center' class='forumheader3'>"; <colgroup span='2'>
$text .= "<input class='button' type='submit' name='news_comments_recalc' value='".LAN_NEWS_57."' /></td></tr>"; <col class='col-label'>
<col class='col-control'>
$text .= "</table></div> </colgroup>
</form>"; <tbody>
<tr>
<td class='label'>".LAN_NEWS_56."</td>
<td class='control'>
".$frm->admin_button('news_comments_recalc', LAN_NEWS_57, 'update')."
</td>
</tr>
</tbody>
</table>
</fieldset>
</form>
";
$emessage = &eMessage::getInstance(); $emessage = &eMessage::getInstance();
$e107->ns->tablerender(LAN_NEWS_59, $emessage->render().$text); $e107->ns->tablerender(LAN_NEWS_59, $emessage->render().$text);

10
e107_handlers/form_handler.php
View File

@@ -9,8 +9,8 @@
* Form Handler * Form Handler
* *
* $Source: /cvs_backup/e107_0.8/e107_handlers/form_handler.php,v $ * $Source: /cvs_backup/e107_0.8/e107_handlers/form_handler.php,v $
* $Revision: 1.23 $ * $Revision: 1.24 $
* $Date: 2009-01-28 14:57:27 $ * $Date: 2009-05-10 17:31:51 $
* $Author: secretr $ * $Author: secretr $
* *
*/ */
@@ -65,6 +65,10 @@ class e_form
var $_tabindex_counter = 0; var $_tabindex_counter = 0;
var $_tabindex_enabled = true; var $_tabindex_enabled = true;
var $_cached_attributes = array(); var $_cached_attributes = array();
/**
* @var user_class
*/
var $_uc; var $_uc;
function e_form($enable_tabindex = false) function e_form($enable_tabindex = false)
@@ -279,8 +283,8 @@ class e_form
function option($option_name, $value, $selected = false, $options = array()) function option($option_name, $value, $selected = false, $options = array())
{ {
if(false === $value) $value = ''; if(false === $value) $value = '';
$options['selected'] = $selected; //comes as separate argument just for convenience
$options = $this->format_options('option', '', $options); $options = $this->format_options('option', '', $options);
$options['selected'] = $selected; //comes as separate argument just for convenience
return "<option value='{$value}'".$this->get_attributes($options).">{$option_name}</option>"; return "<option value='{$value}'".$this->get_attributes($options).">{$option_name}</option>";
} }