Update auf 2.0.23

[bce]

als ich gerade das update einbauen wollte, musste ich feststellen, dass in der admin_styles.php 2 zeilen doppelt vorkommen...

es betrifft die zeilen die mit "$confirm = " und "$cancel = " beginnen...

Code: Alles auswählen

//
// Load default header
//
//
// Check if the user has cancled a confirmation message.
//
$phpbb_root_path = "./../";
require($phpbb_root_path . 'extension.inc');

$confirm = ( isset($HTTP_POST_VARS['confirm']) ) ? TRUE : FALSE;
$cancel = ( isset($HTTP_POST_VARS['cancel']) ) ? TRUE : FALSE;

$no_page_header = (!empty($HTTP_POST_VARS['send_file']) || $cancel) ? TRUE : FALSE;

require('./pagestart.' . $phpEx);

$confirm = ( isset($HTTP_POST_VARS['confirm']) ) ? TRUE : FALSE;
$cancel = ( isset($HTTP_POST_VARS['cancel']) ) ? TRUE : FALSE;

welche der zeilen kann weg? (10+11 oder 17+18)
¢¦


zweite frage:

(betrifft die datei groupcp.php)

in der anleitung steht:

if ( !empty($HTTP_POST_VARS['add']) || !empty($HTTP_POST_VARS['remove']) || isset($HTTP_POST_VARS['approve']) || isset($HTTP_POST_VARS['deny']) )
{
if ( !$userdata['session_logged_in'] )
{
redirect(append_sid("login.$phpEx?redirect=groupcp.$phpEx&" . POST_GROUPS_URL . "=$group_id", true));

#
#-----[ AFTER, ADD ]---------------------------------------------
#
}
else if ( $sid !== $userdata['session_id'] )
{
message_die(GENERAL_ERROR, $lang['Session_invalid']);

muss nicht nach

message_die(GENERAL_ERROR, $lang['Session_invalid']);

noch eine schliessende klammer?

[cback]

zu 1)
Wenn Du die Datei editierst ist immer die nächste Fundstelle von Deiner Cursorposition gemeint. Also wenn Du in der Datei moddest bist am Anfang, dann die erste Fundstelle. Bringt Dich eine Editierung an dieser Stelle vorbei findet der Editor logischerweise die zweite. Dann wäre die gemeint.


zu 2)
Ja, das ergibt sich aber. Die Fundstelle findet ohne das schließende } von dem if, das else if erneuert bis zur öffnenden klammer {, die vom öffnenden if { steht aber nach wie vor drunter.

[oxpus]

Zu 1: Die 2. Zeilen mit $cancel und $confirm können weg. Die sind doppelt.

BTW: Falsches Forum !!!

[Dungeonwatcher]

Hi große Meister!

Das updaten funktionierte doch tatsächlich wieder ohne größere Probleme. THNX an die Macher dieses Updates.

Eine Frage habe ich aber trotzdem:

Wenn ich die update_to_latest.php ausführen will, dann bekomme ich nur den Hinweis zum lesen auf die install.html. Abgesehen davon, das es die nicht gibt, steht in der phpbb-2.0.22_to_2.0.23.txt das ich die update_to_latest.php ausführen soll.

Was also tun?

Bye

Nachtrag:

Ich musste den $phpbb_root_path = './../'; auf $phpbb_root_path = './'; ändern, dann klappte auch der letzte Schritt des Updates.

[oxpus]

Oder die Datei in einen neu zu erstellenden Ordner "install/" hochladen, wie es gedacht ist

[bce]

thx

[drkmnn666]

Code: Alles auswählen

# 
#-----[ OPEN ]--------------------------------------------- 
# 
privmsg.php

#
#-----[ FIND ]---------------------------------------------
# Line 666

	if ( !$confirm )
	{
		$s_hidden_fields = '<input type="hidden" name="mode" value="' . $mode . '" />';
		$s_hidden_fields .= ( isset($HTTP_POST_VARS['delete']) ) ? '<input type="hidden" name="delete" value="true" />' : '<input type="hidden" name="deleteall" value="true" />';

#
#-----[ AFTER, ADD ]---------------------------------------------
#
		$s_hidden_fields .= '<input type="hidden" name="sid" value="' . $userdata['session_id'] . '" />';

#
#-----[ FIND ]---------------------------------------------
# Line 697
		$template->pparse('confirm_body');

		include($phpbb_root_path . 'includes/page_tail.'.$phpEx);

	}

#
#-----[ FIND ]---------------------------------------------
# Line 702
	else if ( $confirm )

#
#-----[ REPLACE WITH ]---------------------------------------------
#
	else if ($confirm && $sid === $userdata['session_id'])

wenn ich das einbaue, dann kommt beim benutzen fehlermeldung, woran könnte das liegen?

[oxpus]

Und wie lautet die Fehlermeldung?

[drkmnn666]

fehler in privmsg.php zeile 700, also genau das, was ich eingebaut bzw. geändert habe

[oxpus]

Die lautet die Fehlermeldung genau?

[drkmnn666]

bin gerade dabei es noch mal einzubauen, poste sie dann unverzüglich

Code: Alles auswählen

Parse error: syntax error, unexpected T_ELSE in /usr/export/www/vhosts/funnetwork/hosting/test/privmsg.php on line 770

[oxpus]

Und wie schaut die Zeile 700 (-/+ 10-15 Zeilen) aus?

[KeineAhnung]

[quote="oxpus";p="84966"]Und wie schaut die Zeile 700 (-/+ 10-15 Zeilen) aus?[/quote]

770... nicht das du ihn noch verwirrst

[drkmnn666]

so hier die ganze datei.... es kommt noch mehr auf euch zu *hehe*

[oxpus]

@Keine Ahnung
Oh ja, 770. Sorry

@drkmnn666
Die Datei ist ungemoddet?!?!?!?
Also für einen Einbau von MODs fehlt mir schlicht die Zeit und/oder ohne Bezahlung auch die Motivation...

[drkmnn666]

logisch, ungemodet ist die nicht, hab das halt nicht geändert, da es funktioniert, denke mal dies ist nicht so tragisch. wenn ich mal viel geld überhabe, so nen 10er, dann komm ich mal auf dich zu mit etwas schwerem... danke trotzdem

[oxpus]

Öhm, Du solltest das phpBB-Update auf jeden Fall vollständig installieren. Nicht umsonst werden an diesen Stellen (wenn auch nicht gerade sofort erkennbar) Sicherheitslücken geschlossen.
Nur als Warnung, falls genau an dieser Stelle bei Dir einer einbricht und Dein Board damit hackt