Meine Probleme mit dem update auf 2.0.18

[Snoopy]

Deine phpBB Version: phpBB2 Plus 1.5.2
MODs: Nein
Dein Wissensstand: Einsteiger
Link zu Deinem Forum: [url]http://[/url]

PHP Version:
MySQL Version:


Was hast Du gemacht, bevor das Problem aufgetreten ist?



Was hast Du bereits versucht um das Problem zu lösen?




Fehlerbeschreibung und Nachricht

Code: Alles auswählen

#
#-----[ OPEN ]---------------------------------------------
#
admin/admin_board.php



#
#-----[ FIND ]---------------------------------------------
#
	"L_VISUAL_CONFIRM_EXPLAIN" => $lang['Visual_confirm_explain'],

#
#-----[ AFTER, ADD ]---------------------------------------------
#
	"L_ALLOW_AUTOLOGIN" => $lang['Allow_autologin'],
	"L_ALLOW_AUTOLOGIN_EXPLAIN" => $lang['Allow_autologin_explain'],
	"L_AUTOLOGIN_TIME" => $lang['Autologin_time'],
	"L_AUTOLOGIN_TIME_EXPLAIN" => $lang['Autologin_time_explain'],

Das gesuchte kann ich nicht finden. Was soll ich denn jetzt tun???

[oxpus]

Füge das einfach nach irgend einem 'L_...' => ein. Hauptsache es steht drinnen.

[modbo]

Code: Alles auswählen

#
# --- [ FIND ] ---
#
   "L_ALLOW_SMILIES" => $lang['Allow_smilies'],

#
# --- [ AFTER, ADD ] ---
#
   "L_ALLOW_AUTOLOGIN" => $lang['Allow_autologin'], 
   "L_ALLOW_AUTOLOGIN_EXPLAIN" => $lang['Allow_autologin_explain'], 
   "L_AUTOLOGIN_TIME" => $lang['Autologin_time'], 
   "L_AUTOLOGIN_TIME_EXPLAIN" => $lang['Autologin_time_explain'],

Oder an einer anderen Stelle, Hauptsache nach

Code: Alles auswählen

$template->assign_vars(array(

und vor

Code: Alles auswählen

...   ])
);

$template->pparse("body");

[Snoopy]

Super danke, dann kommen wir zum nächsten! Ich hab da eine längere Liste!

Code: Alles auswählen

admin/admin_users.php

#
#-----[ FIND ]---------------------------------------------
#
		if( $HTTP_POST_VARS['deleteuser'] )

#
#-----[ REPLACE WITH ]---------------------------------------------
#
		if( $HTTP_POST_VARS['deleteuser'] && ( $userdata['user_id'] != $user_id ) )

Diese Zeile sieht bei mir so aus:

Code: Alles auswählen

if( $HTTP_POST_VARS['deleteuser'] && $new_user==0)

Einfach nur das austauschen was als erstes steht? Ich war mir da nicht sicher!

[modbo]

Ich würde sagen

Code: Alles auswählen

if( $HTTP_POST_VARS['deleteuser'] && $new_user==0 && ( $userdata['user_id'] != $user_id ) )

[Snoopy]

Bist du dir sicher?

[oxpus]

Yepp, vollkommen.

[Snoopy]

Ich tippere alles nach und nach was ich habe, damit ich nicht durcheinander komme.

Also das nächste:

Code: Alles auswählen

includes/bbcode.php

#
#-----[ FIND ]---------------------------------------------
#
				bbcode_array_push($stack, $match);

#
#-----[ REPLACE WITH ]---------------------------------------------
#
				array_push($stack, $match);

Hab ich doppelt, alle beiden austauschen? Wenn noch was doppelt ist auch beides austauschen?

[modbo]

Wenn nicht anders angegeben immer nur das "Erste" (gelesen von oben nach unten) bearbeiten.

[Snoopy]

Ok, hab das erste Ausgetauscht!

Code: Alles auswählen

#-----[ OPEN ]---------------------------------------------
#
includes/functions_post.php

#
#-----[ FIND ]---------------------------------------------
#
		$page_title = $lang['Emoticons'] . " - $topic_title";

#
#-----[ REPLACE WITH ]---------------------------------------------
#
		$page_title = $lang['Emoticons'];
Kann ich nicht finden!

Kann ich nicht machen, da ich das gesuchte nicht finde!

[cbrkiter]

Hallo,

das hab ich bei mir auch nicht gefunden. Aber als ich nach der Variablen $page_title suchte, fand ich

Code: Alles auswählen

$page_title = $lang['Review_topic'] . " - $topic_title"

Warum auch immer dies bei mir anders war. Vielleicht irgendein MOD? Da die Variable sonst nirgends neu einen Wert zugewiesen bekommen hat, habe ich diese Zeile ersetzt. Ich wüsste nichts, was dagegen spricht. Wer anders vielleicht?

[oxpus]

@Snoopy
Wenn bei Dir die Zeile bereits $page_title = $lang['Emoticons']; lautet, ist alles okay.

@cbrkiter
Ist i.O. so. Die Zeile soll ja auch nur "Smilies" herausbringen. Also im Fenstertitel des betreffenden Popups.

[Snoopy]

Bei mir war es so wie bei cbcriter!

Code: Alles auswählen

#
#-----[ OPEN ]---------------------------------------------
#
includes/functions_validate.php

#
#-----[ FIND ]---------------------------------------------
#
		if ($row = $db->sql_fetchrow($result))

#
#-----[ REPLACE WITH ]---------------------------------------------
#
		while ($row = $db->sql_fetchrow($result))

Das gibt es auch mehrmal, mehrmals als 2 mal!

Ich hab jetzt das erste genommen, ist das richtig?

[oxpus]

Grundsätzlich: Immer die erste Fundstelle verwenden. Nur, wenn in der Anleitung was anderes enthalten ist, dann die nächste Stelle auch verwenden.

[Snoopy]

Code: Alles auswählen

includes/sessions.php

#
#-----[ FIND ]---------------------------------------------
#
	$current_time = time();
	$expiry_time = $current_time - $board_config['session_length'];

	//
	// Try and pull the last time stored in a cookie, if it exists
	//
	$sql = "SELECT *
		FROM " . USERS_TABLE . "
		WHERE user_id = $user_id";
	if ( !($result = $db->sql_query($sql)) )
	{
		message_die(CRITICAL_ERROR, 'Could not obtain lastvisit data from user table', '', __LINE__, __FILE__, $sql);
	}

	$userdata = $db->sql_fetchrow($result);

	if ( $user_id != ANONYMOUS )
	{
		$auto_login_key = $userdata['user_password'];

		if ( $auto_create )
		{
			if ( isset($sessiondata['autologinid']) && $userdata['user_active'] )
			{
				// We have to login automagically
				if( $sessiondata['autologinid'] === $auto_login_key )
				{
					// autologinid matches password
					$login = 1;
					$enable_autologin = 1;
				}
				else
				{
					// No match; don't login, set as anonymous user
					$login = 0;
					$enable_autologin = 0;
					$user_id = $userdata['user_id'] = ANONYMOUS;
				
					$sql = 'SELECT * FROM ' . USERS_TABLE . ' WHERE user_id = ' . ANONYMOUS;
					$result = $db->sql_query($sql);
					$userdata = $db->sql_fetchrow($result);
					$db->sql_freeresult($result);
				}
			}
			else
			{
				// Autologin is not set. Don't login, set as anonymous user
				$login = 0;
				$enable_autologin = 0;
				$user_id = $userdata['user_id'] = ANONYMOUS;

				$sql = 'SELECT * FROM ' . USERS_TABLE . ' WHERE user_id = ' . ANONYMOUS;
				$result = $db->sql_query($sql);
				$userdata = $db->sql_fetchrow($result);
				$db->sql_freeresult($result);
			}
		}
		else
		{
			$login = 1;
		}
	}
	else
	{
		$login = 0;
		$enable_autologin = 0;
	}

#
#-----[ REPLACE WITH ]---------------------------------------------
#
	

Was muss ich bei diesem Code machen? Da der Cracker Tracker ja in diesem Abschnitt mit drin ist?

[oxpus]

Ersetzen. Der Cracker Tracker muss in der aktuellen Version nicht mehr in der sessions.php installiert werden.
Übrigens ist auch genau das schon die Änderung vom CrackerTracker 3.1.2 auf 3.1.3

[Snoopy]

Ok, nach diesem update werde ich den CT auf den neusten satnd bringen!

Dann hab ich weitere Probleme:

Code: Alles auswählen

#-----[ OPEN ]---------------------------------------------
#
includes/sessions.php

#
#-----[ FIND ]---------------------------------------------
#
	{// ( $userdata['user_session_time'] > $expiry_time && $auto_create ) ? $userdata['user_lastvisit'] : (

#
#-----[ REPLACE WITH ]---------------------------------------------
#
	{

Das { ist auskommentiert, ist das richtig?

Code: Alles auswählen

#
#-----[ OPEN ]---------------------------------------------
#
includes/sessions.php
#
#-----[ FIND ]---------------------------------------------
#
					//
					// Delete expired sessions
					//
					$expiry_time = $current_time - $board_config['session_length'];

					$sql = "DELETE FROM " . SESSIONS_TABLE . "
						WHERE session_time < $expiry_time
							AND session_id <> '$session_id'";
					if ( !$db->sql_query($sql) )
					{
						message_die(CRITICAL_ERROR, 'Error clearing sessions table', '', __LINE__, __FILE__, $sql);
					}

#
#-----[ REPLACE WITH ]---------------------------------------------
#
					session_clean($userdata['session_id']);

Schaut bei mir anders aus:

Code: Alles auswählen

//
					// Delete expired sessions
					//
					$expiry_time = $current_time - $board_config['session_length'];
					$sql = "DELETE FROM " . SESSIONS_TABLE . " 
						WHERE UNIX_TIMESTAMP() - session_time >=172800
							AND session_id <> '$session_id'";
					if ( !$db->sql_query($sql) )
					{
						message_die(CRITICAL_ERROR, 'Error clearing sessions table', '', __LINE__, __FILE__, $sql);
					}

Kann ich das einfach so ändern?

[modbo]

Mal ne Gegenfrage:
Benutzt du eigentlich die eigens bereitsgestellten Manual Upgrade Instructions for phpBB2 Plus 1.5x?
Dort sind, nach kurzem Blick, jedenfalls die Änderungen (FIND) für die Datei admin/admin_users so angegeben, wie sie im phpBB Plus auch vorkommen.

http://www.phpbb2.de/viewtopic.php?t=33561

[oxpus]

@Snoopy
Unabhängig des Forums:
Die Klammer ist nicht auskommentiert, sondern der Text dahinter.
Und dieser soll ja weg, da er überflüssig ist.

sessions.php: Ersetze diesen Block. Das ist schon okay...

[Snoopy]

@modbo
Nein benutze ich nicht! Ich benutze den Code, den es hier auf oxpus.de gibt!



Wenn ich den Code austauschen tue, komme ich nicht mehr auf mein Board!

Code: Alles auswählen

Fatal error: Call to undefined function: session_clean() in /www/htdocs/w005c6f5/forum/includes/sessions.php on line 366

[oxpus]

Wohl ein paar Funktionen nicht aus dem Update eingefügt?

[Snoopy]

Hast recht mir fehlen noch die Templates:

Code: Alles auswählen

templates/subSilver/groupcp_info_body.tpl
#-----[ FIND ]---------------------------------------------
#
		<td class="catBottom" colspan="8" align="right"><span class="cattitle">
			<input type="submit" name="remove" value="{L_REMOVE_SELECTED}" class="mainoption" />

#
#-----[ REPLACE WITH ]---------------------------------------------
#
		<td class="catBottom" colspan="8" align="right">
			<span class="cattitle"><input type="submit" name="remove" value="{L_REMOVE_SELECTED}" class="mainoption" /></span>

Sieht bei mir so aus:

Code: Alles auswählen

</td>
 		<td align="right"><input type="submit" name="remove" value="{L_REMOVE_SELECTED}" class="mainoption" /></td>
 		</tr>
 	</table>
 
</td>

Kann ich das bedenkenlos änder?

[modbo]

Brauchst du gar nicht ändern, da hier nur ein fehlendes </span> im subsilver Style korrigiert wird.
Nochmal der Hinweis, doch vielleicht die Code Changes für das phpBB Plus zu verwenden!

[Snoopy]

Aber da ist ja nur das drin was ich in der language german ändern muss! Ausserdem hab ich ein paar Mods eingebaut!

Und was ist mit diesem hier:

Code: Alles auswählen

templates/subSilver/index_body.tpl

#
#
# NOTE --- The following action adds the ability to show the 'Mark forums read' link to logged in users only.
#
#-----[ FIND ]---------------------------------------------
#
	<td align="left"><span class="gensmall"><a href="{U_MARK_READ}" class="gensmall">{L_MARK_FORUMS_READ}</a></span></td>

#
#-----[ REPLACE WITH ]---------------------------------------------
#
 	<td align="left">
 	<!-- BEGIN switch_user_logged_in -->
 		<span class="gensmall"><a href="{U_MARK_READ}" class="gensmall">{L_MARK_FORUMS_READ}</a></span>
 	<!-- END switch_user_logged_in -->
 	</td>

Das schaut bei mir so aus:

Code: Alles auswählen

<!-- BEGIN switch_user_logged_in -->
<a href="{U_SEARCH_NEW}">{L_SEARCH_NEW}</a><br />
<a href="{U_MARK_READ}"><strong>{L_MARK_FORUMS_READ}</strong></a>
<!-- END switch_user_logged_in -->

Edit:

Ich bin ja so blind. Ich hab den Code fürs 1.52 etc total übersehen und hab das normale gemacht! Hab ich grad erst gesehen! Ich dachte das da nur das dreiminütige drin ist!

Was mach ich denn nun? Alles nochmal? Oder kann das so bleiben?
Weil ich hab bis jetzt 2 Fehler im Forum und bin mit dem Templates noch nicht fertig weil ich da Probleme hatte! Die werd ich jetzt noch mal durch gehen!

[modbo]

[quote="Snoopy - Do 10.Nov, 2005 09:45";p="48019"]Edit:

Ich bin ja so blind. Ich hab den Code fürs 1.52 etc total übersehen und hab das normale gemacht! Hab ich grad erst gesehen! Ich dachte das da nur das dreiminütige drin ist![/quote]


Allerdings stammt der Fehler "Call to undefined function" mit Sicherheit nicht aus einer Template Datei.
Kontrollier einmal die Änderungen in der includes/session.php. Dort wird die Funktion zumindest durch das Update eingebaut.