"Przypomnienie" - modernizacja, poprawki.

Pomysł  Założony przez  Snake_.

Cześć, wiadomość głownie skierowana do @Cezary Stasiak .


Plugin, jak plugin, ale w sumie nie widzę jego sensu ogólnego. Może z tego wyjść jeszcze coś ciekawego. Tylko trzeba zauważyć kilka 'drobnych' błędów.
  • Dodajesz arkusz CSS, a potem co z nim? Brak kodu, który przy dezaktywacji pluginu usuwa arkusz.
  • W jakim celu dołączasz ten plik? 
    require MYBB_ROOT."/inc/adminfunctions_templates.php";
    Nic z nim nie robisz, nie widzę sensu dodania tej linijki. Z drugiej strony możesz ją zostawić, lecz jest małe 'ale', powinieneś dodać find_replace_templatesets(), aby dodawać zmienną do szablonu od razu po aktywacji pluginu (usuwanie zmiennej masz w funkcji _deactivate(). Przemyśl to.
  • Globalizowanie $mybb koniecznie jest tylko w funkcji reminder(), w pozostałych nigdzie nie używasz tego.
    if ($mybb->settings['reminder_enabled'] == 1){
        
    $reminder '<table border="0" cellspacing="5" cellpadding="0" class="tborder"><p align="center"><tr><td class="annocuement">Ogłoszenie</td></tr>
        </p><tr><td class="remindertext"><p align="center">'
    .$mybb->settings['reminder_text'].'</p></td></tr></span></table><br /><br />';
        }
        
        
    $reminder .= '</td></tr></table>';
        } 
    Jeśli jest włączony, to pokazuje się tabelka, jeśli wyłączony - niepotrzebny kod HTML jest w źródle strony. Tak wywnioskowałem z kodu, nie sprawdzałem. W ogóle kod html jest poniekąd dziwny, gdzie użyłeś <span>, skoro zamykasz? Zamknąłeś <table>, w jakim celu jest linijka $reminder .= [...]?

    Cała funkcja:
    function reminder(){
        global 
    $mybb$reminder;
            if(
    $mybb->settings['reminder_enabled'] == 1)
            {
                
    $reminder '
                    <table border="0" cellspacing="5" cellpadding="0" class="tborder">
                        <tr>
                            <td class="annocuement"><p align="center">Ogłoszenie</p></td>
                        </tr>
                        <tr>
                            <td class="remindertext"> <p align="center">'
    .$mybb->settings['reminder_text'].'</p></td>
                        </tr>
                    </table><br /><br />'
    ;
            }

Dodatkowo, jeśli mogę coś zasugerować. Z tego jeszcze można coś ciekawego zrobić, przykłady:
  • Ogłoszenie z poziomu MCP;
  • Uprawnienia grup (widoczność itd.)
  • Ogłoszenie dla poszczególnych grup
  • Edycja z poziomu index'u dla administratorów.
  • i sporo innych myśli.
Z tego wyjdzie coś ciekawego, tylko trzeba przysiąść i popracować. :)
Prośby na PW dotyczące wsparcia z problemami będą ignorowane. Pomoc poza forum - odpłatna; kontakt: snakemybboard@gmail.com.
Dziękuję, od razu dodałeś mi "kopa" do pracy nad pluginem ;)
Po niedługim przemyśleniu fajną kwestią byłoby zarządzanie treścią z poziomu indexu. Jeśli zaciekawi Cię pomysł to jakieś materiały poszukam i podeślę. :)
Prośby na PW dotyczące wsparcia z problemami będą ignorowane. Pomoc poza forum - odpłatna; kontakt: snakemybboard@gmail.com.
Fajny pomysł, dzięki, że poszukasz materiały ;)
Dla innych, skoro zbanowany użytkownik:

Możliwości jest od groma:

JSON / AJAX / Operowanie na bazie danych / itd.
Prośby na PW dotyczące wsparcia z problemami będą ignorowane. Pomoc poza forum - odpłatna; kontakt: snakemybboard@gmail.com.
(29.10.2016, 10:39)Snake_ napisał(a): Możliwości jest od groma:

JSON / AJAX / Operowanie na bazie danych / itd.

Pewnie, że jest - ale wszystko o czym pisałeś ma zastosowanie do jakiegokolwiek pluginu. Plugin Ceza to zwykłe wyświetlanie html'a zapisanego w zmiennej, nic mniej nic więcej :P
"Try not. Do... or do not. There is no try."
Poświęć 5 minut. Nie bądź ignorantem!  -  Jak zbadać element?
@ficus, napisałem to w pierwszym poście. Dlatego można zrobić z tego coś ciekawego, wystarczy chęć i trochę myślenia. :)
Prośby na PW dotyczące wsparcia z problemami będą ignorowane. Pomoc poza forum - odpłatna; kontakt: snakemybboard@gmail.com.



Użytkownicy przeglądający ten wątek:

1 gości