some bugfixes

git-svn-id: file:///svn/phpbb/trunk@8168 89ea8834-ac86-4346-8a33-228a782c2dd0
This commit is contained in:
Meik Sievertsen 2007-10-12 18:14:28 +00:00
parent e365b6abfa
commit d0f2399a86
9 changed files with 40 additions and 32 deletions

View file

@ -117,6 +117,7 @@
<li>[Fix] Match custom BBCodes in the same way during first and second pass - patch provided by IBBoard (Bug #14268)</li> <li>[Fix] Match custom BBCodes in the same way during first and second pass - patch provided by IBBoard (Bug #14268)</li>
<li>[Fix] Correct quote parsing if opening bracket before opening quote (Bug #14667)</li> <li>[Fix] Correct quote parsing if opening bracket before opening quote (Bug #14667)</li>
<li>[Fix] Clean post message for checking length to prevent posting empty messages</li> <li>[Fix] Clean post message for checking length to prevent posting empty messages</li>
<li>[Fix] Display jumpbox if needed for functionality (Bug #14702)</li>
</ul> </ul>
<a name="v30rc4"></a><h3>1.ii. Changes since 3.0.RC4</h3> <a name="v30rc4"></a><h3>1.ii. Changes since 3.0.RC4</h3>

View file

@ -329,7 +329,7 @@ parse_css_file = {PARSE_CSS_FILE}
{ {
// Save CSS contents // Save CSS contents
$sql_ary = array( $sql_ary = array(
'theme_mtime' => @filemtime("{$phpbb_root_path}styles/{$theme_row['theme_path']}/theme/stylesheet.css"), 'theme_mtime' => (int) filemtime("{$phpbb_root_path}styles/{$theme_row['theme_path']}/theme/stylesheet.css"),
'theme_data' => $this->db_theme_data($theme_row) 'theme_data' => $this->db_theme_data($theme_row)
); );
@ -1307,10 +1307,10 @@ parse_css_file = {PARSE_CSS_FILE}
} }
$sql_ary = array( $sql_ary = array(
'image_filename' => $imgfilename, 'image_filename' => (string) $imgfilename,
'image_width' => $imgwidth, 'image_width' => (int) $imgwidth,
'image_height' => $imgheight, 'image_height' => (int) $imgheight,
'image_lang' => $imglang, 'image_lang' => (string) $imglang,
); );
// already exists // already exists
@ -1325,7 +1325,7 @@ parse_css_file = {PARSE_CSS_FILE}
else if (!$imageset_data_row) else if (!$imageset_data_row)
{ {
$sql_ary['image_name'] = $imgname; $sql_ary['image_name'] = $imgname;
$sql_ary['imageset_id'] = $imageset_id; $sql_ary['imageset_id'] = (int) $imageset_id;
$db->sql_query('INSERT INTO ' . STYLES_IMAGESET_DATA_TABLE . ' ' . $db->sql_build_array('INSERT', $sql_ary)); $db->sql_query('INSERT INTO ' . STYLES_IMAGESET_DATA_TABLE . ' ' . $db->sql_build_array('INSERT', $sql_ary));
} }
@ -2158,10 +2158,10 @@ parse_css_file = {PARSE_CSS_FILE}
case 'style': case 'style':
$sql_ary += array( $sql_ary += array(
'template_id' => $template_id, 'template_id' => (int) $template_id,
'theme_id' => $theme_id, 'theme_id' => (int) $theme_id,
'imageset_id' => $imageset_id, 'imageset_id' => (int) $imageset_id,
'style_active' => $style_active, 'style_active' => (int) $style_active,
); );
break; break;
@ -2438,11 +2438,11 @@ parse_css_file = {PARSE_CSS_FILE}
// We could do this using extended inserts ... but that could be one // We could do this using extended inserts ... but that could be one
// heck of a lot of data ... // heck of a lot of data ...
$sql_ary = array( $sql_ary = array(
'template_id' => $style_id, 'template_id' => (int) $style_id,
'template_filename' => "$pathfile$file", 'template_filename' => "$pathfile$file",
'template_included' => (isset($includes[$file])) ? implode(':', $includes[$file]) . ':' : '', 'template_included' => (isset($includes[$file])) ? implode(':', $includes[$file]) . ':' : '',
'template_mtime' => filemtime("{$phpbb_root_path}styles/$template_path$pathfile$file"), 'template_mtime' => (int) filemtime("{$phpbb_root_path}styles/$template_path$pathfile$file"),
'template_data' => file_get_contents("{$phpbb_root_path}styles/$template_path$pathfile$file"), 'template_data' => (string) file_get_contents("{$phpbb_root_path}styles/$template_path$pathfile$file"),
); );
if ($mode == 'insert') if ($mode == 'insert')
@ -2984,10 +2984,10 @@ parse_css_file = {PARSE_CSS_FILE}
$sql_ary = array( $sql_ary = array(
'style_name' => $name, 'style_name' => $name,
'style_copyright' => $copyright, 'style_copyright' => $copyright,
'style_active' => $active, 'style_active' => (int) $active,
'template_id' => $style_row['template_id'], 'template_id' => (int) $style_row['template_id'],
'theme_id' => $style_row['theme_id'], 'theme_id' => (int) $style_row['theme_id'],
'imageset_id' => $style_row['imageset_id'], 'imageset_id' => (int) $style_row['imageset_id'],
); );
$sql = 'INSERT INTO ' . STYLES_TABLE . ' $sql = 'INSERT INTO ' . STYLES_TABLE . '
@ -3115,7 +3115,7 @@ parse_css_file = {PARSE_CSS_FILE}
$sql_ary += array( $sql_ary += array(
'theme_storedb' => $store_db, 'theme_storedb' => $store_db,
'theme_data' => ($store_db) ? $this->db_theme_data($sql_ary, false, $root_path) : '', 'theme_data' => ($store_db) ? $this->db_theme_data($sql_ary, false, $root_path) : '',
'theme_mtime' => filemtime("{$phpbb_root_path}styles/$path/theme/stylesheet.css") 'theme_mtime' => (int) filemtime("{$phpbb_root_path}styles/$path/theme/stylesheet.css")
); );
break; break;
@ -3167,7 +3167,7 @@ parse_css_file = {PARSE_CSS_FILE}
$image_height = $image_width = 0; $image_height = $image_width = 0;
} }
if (strpos($key, 'img_') === 0&& $image_filename) if (strpos($key, 'img_') === 0 && $image_filename)
{ {
$key = substr($key, 4); $key = substr($key, 4);
if (in_array($key, $imageset_definitions)) if (in_array($key, $imageset_definitions))
@ -3175,9 +3175,9 @@ parse_css_file = {PARSE_CSS_FILE}
$sql_ary = array( $sql_ary = array(
'image_name' => $key, 'image_name' => $key,
'image_filename' => str_replace('{PATH}', "styles/$path/imageset/", trim($image_filename)), 'image_filename' => str_replace('{PATH}', "styles/$path/imageset/", trim($image_filename)),
'image_height' => $image_height, 'image_height' => (int) $image_height,
'image_width' => $image_width, 'image_width' => (int) $image_width,
'imageset_id' => $id, 'imageset_id' => (int) $id,
'image_lang' => '', 'image_lang' => '',
); );
$db->sql_query('INSERT INTO ' . STYLES_IMAGESET_DATA_TABLE . ' ' . $db->sql_build_array('INSERT', $sql_ary)); $db->sql_query('INSERT INTO ' . STYLES_IMAGESET_DATA_TABLE . ' ' . $db->sql_build_array('INSERT', $sql_ary));
@ -3223,9 +3223,9 @@ parse_css_file = {PARSE_CSS_FILE}
$sql_ary = array( $sql_ary = array(
'image_name' => $image_name, 'image_name' => $image_name,
'image_filename' => $image_filename, 'image_filename' => $image_filename,
'image_height' => $image_height, 'image_height' => (int) $image_height,
'image_width' => $image_width, 'image_width' => (int) $image_width,
'imageset_id' => $id, 'imageset_id' => (int) $id,
'image_lang' => $row['lang_dir'], 'image_lang' => $row['lang_dir'],
); );
$db->sql_query('INSERT INTO ' . STYLES_IMAGESET_DATA_TABLE . ' ' . $db->sql_build_array('INSERT', $sql_ary)); $db->sql_query('INSERT INTO ' . STYLES_IMAGESET_DATA_TABLE . ' ' . $db->sql_build_array('INSERT', $sql_ary));

View file

@ -99,11 +99,12 @@ function gen_sort_selects(&$limit_days, &$sort_by_text, &$sort_days, &$sort_key,
/** /**
* Generate Jumpbox * Generate Jumpbox
*/ */
function make_jumpbox($action, $forum_id = false, $select_all = false, $acl_list = false) function make_jumpbox($action, $forum_id = false, $select_all = false, $acl_list = false, $force_display = false)
{ {
global $config, $auth, $template, $user, $db; global $config, $auth, $template, $user, $db;
if (!$config['load_jumpbox']) // We only return if the jumpbox is not forced to be displayed (in case it is needed for functionality)
if (!$config['load_jumpbox'] && $force_display === false)
{ {
return; return;
} }

View file

@ -86,7 +86,7 @@ function mcp_forum_view($id, $mode, $action, $forum_info)
} }
} }
make_jumpbox($url . "&amp;i=$id&amp;action=$action&amp;mode=$mode", $forum_id . (($merge_select) ? $selected_ids : ''), false, 'm_'); make_jumpbox($url . "&amp;i=$id&amp;action=$action&amp;mode=$mode" . (($merge_select) ? $selected_ids : ''), $forum_id, false, 'm_', true);
$topics_per_page = ($forum_info['forum_topics_per_page']) ? $forum_info['forum_topics_per_page'] : $config['topics_per_page']; $topics_per_page = ($forum_info['forum_topics_per_page']) ? $forum_info['forum_topics_per_page'] : $config['topics_per_page'];

View file

@ -266,7 +266,7 @@ function mcp_front_view($id, $mode, $action)
} }
$template->assign_var('S_MCP_ACTION', append_sid("{$phpbb_root_path}mcp.$phpEx")); $template->assign_var('S_MCP_ACTION', append_sid("{$phpbb_root_path}mcp.$phpEx"));
make_jumpbox(append_sid("{$phpbb_root_path}mcp.$phpEx", 'i=main&amp;mode=forum_view'), 0, false, 'm_'); make_jumpbox(append_sid("{$phpbb_root_path}mcp.$phpEx", 'i=main&amp;mode=forum_view'), 0, false, 'm_', true);
} }
?> ?>

View file

@ -93,7 +93,7 @@ function mcp_topic_view($id, $mode, $action)
} }
// Jumpbox, sort selects and that kind of things // Jumpbox, sort selects and that kind of things
make_jumpbox($url . "&amp;i=$id&amp;mode=forum_view", $topic_info['forum_id'], false, 'm_'); make_jumpbox($url . "&amp;i=$id&amp;mode=forum_view", $topic_info['forum_id'], false, 'm_', true);
$where_sql = ($action == 'reports') ? 'WHERE post_reported = 1 AND ' : 'WHERE'; $where_sql = ($action == 'reports') ? 'WHERE post_reported = 1 AND ' : 'WHERE';
$sort_days = $total = 0; $sort_days = $total = 0;

View file

@ -117,7 +117,7 @@ $lang = array_merge($lang, array(
'CONV_SAVED_MESSAGES' => 'Saved messages', 'CONV_SAVED_MESSAGES' => 'Saved messages',
'COULD_NOT_COPY' => 'Could not copy file <strong>%1$s</strong> to <strong>%2$s</strong><br /><br />Please check that the target directory exists and is writable by the webserver.', 'COULD_NOT_COPY' => 'Could not copy file <strong>%1$s</strong> to <strong>%2$s</strong><br /><br />Please check that the target directory exists and is writable by the webserver.',
'COULD_NOT_FIND_PATH' => 'Could not find path to your former board. Please check your settings and try again.<br />» Specified source path was %s.', 'COULD_NOT_FIND_PATH' => 'Could not find path to your former board. Please check your settings and try again.<br />» %s was specified as the source path.',
'DBMS' => 'Database type', 'DBMS' => 'Database type',
'DB_CONFIG' => 'Database configuration', 'DB_CONFIG' => 'Database configuration',

View file

@ -12,7 +12,7 @@
<fieldset> <fieldset>
<dl class="fields2"> <dl class="fields2">
<dt><label for="reason_id">{L_REASON}:</label></dt> <dt><label for="reason_id">{L_REASON}:</label></dt>
<dd><select name="reason_id" id="reason_id"><!-- BEGIN reason --><option value="{reason.ID}"<!-- IF reason.S_SELECTED --> selected="selected"<!-- ENDIF -->>{reason.DESCRIPTION}</option><!-- END reason --></select></dd> <dd><select name="reason_id" id="reason_id" class="full"><!-- BEGIN reason --><option value="{reason.ID}"<!-- IF reason.S_SELECTED --> selected="selected"<!-- ENDIF -->>{reason.DESCRIPTION}</option><!-- END reason --></select></dd>
</dl> </dl>
<!-- IF S_CAN_NOTIFY --> <!-- IF S_CAN_NOTIFY -->
<dl class="fields2"> <dl class="fields2">

View file

@ -376,3 +376,9 @@ input.search {
background-position: left 1px; background-position: left 1px;
padding-left: 17px; padding-left: 17px;
} }
.full { width: 95%; }
.medium { width: 50%;}
.narrow { width: 25%;}
.tiny { width: 10%;}