phpBB2 by Przemo

Woytec · **Pomógł:** 2596 razy Posty: 7676

1. Obsługa baz danych
W wersji 1.12.8 wprowadzono obsługę mysqli, domyślnie włączoną. W związku z tym niektóre modyfikacje nie działają, ponieważ wykorzystują nadal funkcje służące do operowania na połączeniu mysql. Gdyby modyfikacje były pisane wg. standardu - tzn. wykorzystywały klasę "db" to problemu by nie było, ponieważ działałyby praktycznie niezależnie od tego, z jakiego silnika bazy danych korzysta pozostała część skryptu (problem może ewentualnie wystąpić z zapytaniami ale w przypadku mysqli to nadal jest taki sam standard jak mysql, to tylko sposób połączenia).
Charakterystyczne komunikaty o błędach zaczynają się na:

Kod:

Warning: mysql****

wskazujące na linię z funkcją dokonującą operacji w bazie danych(np. wywołanie zapytania)

W takim przypadku, w zmodyfikowanych plikach, jak i dołączonych do modyfikacji należy zmienić te funkcje na odpowiedniki z klasy "db".

mysql_query na $db->sql_query
mysql_fetch_array na $db->sql_fetchrow
mysql_num_rows na $db->sql_numrows

2. Cache
W związku ze zmianą sposobu działania cache, mogą pojawić się komunikaty o błędach zaczynające się od:

Kod:

Warning: include(Array)

wskazujące na linię

Kod:

include ($check_cache);

a kilka linii wcześniej

Kod:

sql_cache('check', 'nazwa');

Nowy sposób od razu zwraca zawartość cache a czy coś tam jest sprawdzamy czy zmienna jest pusta a nie jak dotychczas - funkcja zwracała informację o tym, że coś jest w cache i jeżeli zwróciła prawdę to includowaliśmy ten plik.

Nowy wzór:

Kod:

$nazwa = sql_cache('check', 'nazwa');

if(empty($nazwa)
{
//Nie ma nic w cache. Pobieramy z bazy
}
//dalsze operacje

Dla kompatybilności "nazwa" powinna być taka sama w nazwie zmiennej i drugiego parametru funkcji, tak jak we wcześniejszym fragmencie.

3. Funkcja htmlspecialchars
Od wersji php 5.4 zmieniona została zawartość parametru "kodowanie" tej funkcji na nie używane na razie UTF co powodowało problemy m.in przy wysyłaniu postów. Najłatwiej będzie zamienić wszystkie htmlspecialchars w plikach modyfikacji i fragmentach do wyszukania na xhtmlspecialchars - w 1.12.8 stworzyliśmy tę funkcję aby podawała prawidłowy trzeci parametr.

4. Optymalizacje kodu
Nie powoduje to błędów ale ulepsza kod.
Zachęcam do korzystania z funkcji get_vars

Kod:

function get_vars($name, $empty = false, $methods = 'POST,GET', $int = false, $array_level = 0)

Przykład: Zamiast

Kod:

if( isset($HTTP_GET_VARS['mark']) || isset($HTTP_POST_VARS['mark']) )
{
$mark_read = ( isset($HTTP_POST_VARS['mark']) ) ? $HTTP_POST_VARS['mark'] : $HTTP_GET_VARS['mark'];
}
else
{
$mark_read = '';
}

można zapisać

Kod:

$mark_read = get_vars('mark', false, 'POST,GET');

Gadatliwa Kasia

yukos · Posty: 1

Mam pytanie jeśli chodzi o podpunkt 4 - gdzie i jak to mam zrobić... ?
Bardzo proszę o odpowiedź. Pozdrawiam.

Woytec · **Pomógł:** 2596 razy Posty: 7676

Do działania modyfikacji nie jest to ważne. Tworzysz jakąś?

HYPERSHOCKK^ · **Pomógł:** 1 raz Posty: 87

Pkt 2 też nas nie interesuje?

Jak będzie taki komunikat to jakie pliki trzeba edytować aby pozbyć się tego błędu? Bo chyba nie wszystkie załączone z modem i wszystkie jakie edytowaliśmy dla tego moda?

Woytec · **Pomógł:** 2596 razy Posty: 7676

Kajtek223 · Posty: 8

Pomógłbyś mi? Robię punkt 2, ale coś mi nie wychodzi, bo wyskakują wtedy inne błędy.
U mnie jest takie kodowanie, gdzie i jak wstawić? Jakbyś mógł to podstaw mi

Kod:

{
if ($check_cache = sql_cache('check', 'sb_count'))
{
include($check_cache);
$row = $sb_count;
}
else
{

Dzięki

EDIT: Zauważyłem, że był niepotrzebny znak "{" u mnie. Zrobione. Przepraszam za kłopot.

xaxaxa1 · **Pomógł:** 2 razy Posty: 11

Cytat:

Nie powoduje to błędów ale ulepsza kod.

Ulepsza? Wciaz ta funkcja, jak i znaczna wiekszosc zawiera w sobie archaiczny kod z czasow PHP4 ktory w obecnych czasach nie zasluguje nawet na miano slowa "tandentny".

Jak chcesz coś ulepszyć to zamiast

Kod:

if( isset($HTTP_GET_VARS['mark']) || isset($HTTP_POST_VARS['mark']) )
{
$mark_read = ( isset($HTTP_POST_VARS['mark']) ) ? $HTTP_POST_VARS['mark'] : $HTTP_GET_VARS['mark'];
}
else
{
$mark_read = '';
}

sens ma jedynie raczej coś takiego:

Kod:

$mark_read = (isset($_REQUEST['mark']) && !empty($_REQUEST['mark'])) ? $_REQUEST['mark'] : '';

Skoro wydajecie już "nowe wersje" to wysilcie się choć trochę i poprawcie kod sprzed kilku lat - wystarczy odpalić "przema" na serwerze z PHP 5.3 / 5.4 + error_reporting(E_ALL);

Tommekk80 · **Pomógł:** 1411 razy Posty: 4780

$_POST i $_GET pozwala na szybsze i łatwiejsze zlokalizowanie ewentualnego problemu. $_REQUEST już nie - mamy go używać bo co - bo jest?

NineX · **Pomógł:** 546 razy Posty: 1983

xaxaxa1 · **Pomógł:** 2 razy Posty: 11

Gadatliwa Kasia

Woytec · **Pomógł:** 2596 razy Posty: 7676