Fehlermeldung in der viewonline.php

[4seven]

Du hast den Mod Block in den reply und quote Case Teil zusätzlich drin.
Den Block rausnehmen und für diesen Bereich den Originalzustand herstellen:

Code: Select all

                            case 'reply':
                            case 'quote':
                                // BEGIN: Topic in "Who is online"     

Anregung: https://www.phpbb.com/customise/db/mod/ ... 35#p243635

[Tim]

Was soll ich rausnehmen?

[4seven]

Ersetze das

Code: Select all

                        switch ($posting_mode)
                        {
                            case 'reply':
                            case 'quote':
                                // BEGIN: Topic in "Who is online"
                                if ($auth->acl_get('a_')){
                                preg_match('#t=([0-9]+)#', $row['session_page'], $on_page);
                                preg_match('#p=([0-9]+)#', $row['session_page'], $on_page_p);
                                if (sizeof($on_page) && isset($topic_titles[$on_page[1]]))
                                {
                                    $location = sprintf($user->lang['REPLYING_MESSAGE_TOPIC'], $forum_data[$forum_id]['forum_name'], $topic_titles[$topic_post_ids[$on_page[1]]]); 
                                    $location_url = append_sid("{$phpbb_root_path}viewtopic.$phpEx", 'f=' . $forum_id . '&t=' . $on_page[1]);
                                }
                                else if (sizeof($on_page_p) && isset($topic_post_ids[$on_page_p[1]]))
                                {
                                    $location = sprintf($user->lang['REPLYING_MESSAGE_TOPIC'], $forum_data[$forum_id]['forum_name'], $topic_titles[$topic_post_ids[$on_page[1]]]);
                                    $location_url = append_sid("{$phpbb_root_path}viewtopic.$phpEx", 'f=' . $forum_id . '&p=' . $on_page_p[1]);
                                }
                                else
                                {
                                    $location = sprintf($user->lang['REPLYING_MESSAGE'], $forum_data[$forum_id]['forum_name']);
                                }
                                }
                                else{
                                $location = sprintf($user->lang['REPLYING_MESSAGE'], $forum_data[$forum_id]['forum_name']);
                                }
                                // END: Topic in "Who is online"
                            break;

                            default:
                                $location = sprintf($user->lang['POSTING_MESSAGE'], $forum_data[$forum_id]['forum_name']);
                            break;
                        } 

mit dem

Code: Select all

                        switch ($posting_mode)
                        {
                            case 'reply':
                            case 'quote':
                                $location = sprintf($user->lang['REPLYING_MESSAGE'], $forum_data[$forum_id]['forum_name']);
                            break;

                            default:
                                $location = sprintf($user->lang['POSTING_MESSAGE'], $forum_data[$forum_id]['forum_name']);
                            break;
                        } 

[Tim]

Dann ist aber die Funktion weg, dass nur der Admin sehen kann, wer welchen Beitrag liest.

[4seven]

Nein, der Fehler an dem Mod ist, das zusätzlich der Posting Vorgang mit einbezogen ist, hier aber Parameter fehlen (Index Array). Deswegen die Meldung mit dem fehlenden Index. Für das besuchen der viewtopic bleibt alles wie gehabt, da der Mod die viewtopic ja noch an zwei Stellen in der Datei bedient. Die wichtige ist hier:

Code: Select all

                    case 'viewtopic':
                        // BEGIN: Topic in "Who is online"
                        if ($auth->acl_get('a_')){
                        preg_match('#t=([0-9]+)#', $row['session_page'], $on_page);
                        preg_match('#p=([0-9]+)#', $row['session_page'], $on_page_p);
                        if (sizeof($on_page) && isset($topic_titles[$on_page[1]]))
                        {
                            $location = sprintf($user->lang['READING_TOPIC_TOPIC'], $forum_data[$forum_id]['forum_name'], $topic_titles[$on_page[1]]);
                            $location_url = append_sid("{$phpbb_root_path}viewtopic.$phpEx", 'f=' . $forum_id . '&t=' . $on_page[1]);
                        }
                        else if (sizeof($on_page_p) && isset($topic_post_ids[$on_page_p[1]]))
                        {
                            $location = sprintf($user->lang['READING_TOPIC_TOPIC'], $forum_data[$forum_id]['forum_name'], $topic_titles[$topic_post_ids[$on_page_p[1]]]);
                            $location_url = append_sid("{$phpbb_root_path}viewtopic.$phpEx", 'f=' . $forum_id . '&p=' . $on_page_p[1]);
                        }
                        else
                        {
                            $location = sprintf($user->lang['READING_TOPIC'], $forum_data[$forum_id]['forum_name']);
                        }
                        }
                        else{
                        $location = sprintf($user->lang['READING_TOPIC'], $forum_data[$forum_id]['forum_name']);
                        }
                        // END: Topic in "Who is online"
                    break;

                    case 'viewforum':
                        $location = sprintf($user->lang['READING_FORUM'], $forum_data[$forum_id]['forum_name']);
                    break; 

Wie schon gesagt, der Mod müsste überarbeitet werden, da sind einige Fehler drin.
Um die zu lösen, sollte der Autor den Support liefern. Die Version ist immer noch für phpbb 3.0.2
https://www.phpbb.com/customise/db/mod/ ... is_online/

Indexproblem
https://www.phpbb.com/customise/db/mod/ ... ex-t_97915

Vielleicht schau ich mir das mal an, im Moment aber keine Zeit dafür.

[Tim]

Achso, ich verstehe. Ist auch klar, dass du mit deinen MODs genug zu tun hast, trotzdem danke ich dir
für´s Probieren.

[4seven]

Naja, selbst für die ist zur Zeit wenig übrig. Demnächst hab ich dann mal wieder mehr. Die Prioritäten sind ja klar. Erst die Arbeit, dann das Vergnügen. So long..