Poprzedni temat :: Następny temat |
paszczak000 Wysłany: Wto 04 Lip, 2006 |
|
|
Cytat(Wi?cej wida? na screenach).
popraw posta to b?dzie wida? :] |
|
KnofeL Wysłany: Wto 04 Lip, 2006 |
|
|
paszczak000, zobacz co si? dzieje jak dodam screeny :/
[ Dodano: Wto 04 Lip, 2006 ]
Dobra doda?em te obrazki do CODE :/ |
|
kwiatek Wysłany: Sro 05 Lip, 2006 |
|
|
fajny bajer wypr?buje go
gdzie w PA si? go konfiguruje :?: |
|
Szpakenhoffen Wysłany: Sob 08 Lip, 2006 |
|
|
Gdzie to jest? KnofeL napisał/aKod
http://img144.imagevenue....mod_f_292lo.jpg
Nie moge znale?c mam w panelu admina tylko edycje pytan i kandytat?w ale gdzie si? pojawia formularz zg?oszeniowy? Mo?e gdzie? pope?ni?em b??d? |
|
marcinn12 Wysłany: Sob 08 Lip, 2006 |
|
|
Ja dopiero znalaz?em jak wpisa?em w przegladarce... ale tego w Pa nie ma chyba.
http://siemensandse.1stwe...otification.php
Szpakenhoffen, Zr?b tak adresforum/notification.php i bedziesz m?g? dodac;/
EDIT:
Mam inny problem wype?niam wszystkie pola i mi wywala b?ad musisz wype?nic wszystkie pola... bankowo we wszystkich wpisuje info!! Czym to jest spowodowane?? |
|
PiotrLegnica Wysłany: Sob 08 Lip, 2006 |
|
|
Lepszy od tamtego "moda", ale:
- u?ywaj szablon?w [nie wiem czemu, ale co? mnie trafia jak widz? PHP wymieszany z HTMLem :)]
- czemu PHP Code:<?php
for($x=0; $x<=$ile_pytan-1; $x++)
?> zamiast po prostu PHP Code:<?php
for($x=0; $x<$ile_pytan; $x++)
?> - IMO czytelniejsze
- PHP Code:<?php
<LINK REL=stylesheet HREF="templates/Akagahara/Akagahara.css" TYPE=text/css>
?> na PHP Code:<?php
<link rel="stylesheet" href="templates/Akagahara/Akagahara.css" type="text/css" />
?> bo widz?, ?e w XHTML piszesz (i brakuje DOCTYPE oraz title).
- niepotrzebnie stosujesz " " dla string?w. ' ' jest szybsze (PHP wy?wietla jak leci, nie parsuje stringu w poszukiwaniu zmiennych/znak?w specjalnych).
- niepotrzebnie powtarzasz getcwd() za ka?dym razem - zr?b np. PHP Code:<?php
$cwd = getcwd();
?> i u?ywaj tej zmiennej (w obr?bie jednego skryptu katalog roboczy si? nie zmienia, chyba, ?e o czym? nie wiem :)).
- PHP Code:<?php
<tag param='tekst'>
?> zamiast PHP Code:<?php
<tag param="tekst">
?>
- PHP Code:<?php
$data = date('d.m.Y')." ".date('G:i');
?> - niepotrzebne wywo?anie dwa razy tej samej funkcji [zmienna zreszt? te? niepotrzebna], zr?b PHP Code:<?php
$data = date('d.m.Y G:i');
?>
- W notification.php jest niezamkni?ty dokument (brakuje </body></html>)
- W cm_mod_notifications.php brakuje <table></table>
- 0 sprawdzania uprawnie? - ka?dy mo?e zmieni?/usun??/doda? pytania czy zg?oszenia (wystarczy zna? nazw? pliku) [chyba, ?e co? przeoczy?em to sorry]
- Pytania IMO lepiej przechowywa? w bazie danych.
- [php0]foreach ($lista_uczestnikow as $num => $name) { $last[$num] = $name; }[/php0] Niepotrzebne przepisanie tablicy. To samo mo?esz osi?gn?? pisz?c [php1]$last = $lista_uczestnikow[/php1]
- [php2]for($x=0;$x<=count($pytanie)-1;$x++)[/php2] - count jest wywo?ywany przy ka?dej iteracji, zr?b [php3]for($x=0,$y=count($pytanie);$x<$y;$x++)[/php3]
- [php4]$user = getcwd()."/zgloszenia/".$_GET['user'].".usr";
$odpowiedz = file($user);[/php4] - niepotrzebna zmienna tymczasowa
Tyle wy?apa?em po przejrzeniu skryptu :) Jeszcze p??niej go dok?adniej przeanalizuj |
|
KnofeL Wysłany: Sob 08 Lip, 2006 |
|
|
PiotrLegnica, Dzi?ki!
Wszytko to zostanie uwzgl?dnione w przysz?ej wersji. (Kt?ra powinna by? ju? w poniedzia?ek)
Co do daty mia?em tak zrobione ale jako? nie dzia?a?o mi to.. Ale przyjrze si? temu jeszcze raz
Co to uprawnie? wiem w?asnie, tylko jeszcze nie wiem jak to zorbi? zeby dzia?a?o z logowaniem do PA.. ale te? pomysle nad tym.
Co do CytatW cm_mod_notifications.php brakuje <table></table> Owszem nie ma bo plik ten jest includowany i <table></table> jest w admin_cmod.php (Nie wiem czy jest to wa?ne ale je?li tak to zmienie to)
I zgadzam sie co do u?ycia bazy danych, niestety moja wiedza na ten temat jest r?wna zeru wi?c p?ki co nic z tego. Chyba ?e ktos b?dzie w stanie pom?c.
Dzi?ki jeszcze raz, masz za to piwko i czekam na dalesz b?edy :D |
|
PiotrLegnica Wysłany: Sob 08 Lip, 2006 |
|
|
KnofeL napisał/aOwszem nie ma bo plik ten jest includowany i <table></table> jest w admin_cmod.php (Nie wiem czy jest to wa?ne ale je?li tak to zmienie to)
Ah, teraz dopiero to zauwa?y?em :) Ale w takim razie w cm_mod_notifications.php nie powinno by? niczego opr?cz tre?ci tabeli czyli wywali?:
PHP Code:<?php
<html>
<head>
<meta http-equiv="Content-type" content="text/html; charset=iso-8859-2" />
<LINK REL=stylesheet HREF=" include('styl.stl'); " TYPE=text/css>
</head>
<body>
?>
i
PHP Code:<?php
</body>
</html>
?>
I admin_cmod.php tak?e jest niezamkni?ty (brakuje </body></html>). |
|
marcinn12 Wysłany: Nie 09 Lip, 2006 |
|
|
KnofeL, dlaczego jak wype?niam ankiete ca?? i daja napis wyskakuje ze nie wszystkie pola sa wype?nione a bankowo s? i nie chce mi zatwierdzic!! Co to moze byc?? |
|
KnofeL Wysłany: Nie 09 Lip, 2006 |
|
|
M?j b?ad dzisiaj to zauwa?y?em, nie wiem jak to przesz?o testy :oops:
Problem juz rozwi?za?em ale b?dziesz musia? poczeka? do jutra na now? wersje sorry. |
|
marcinn12 Wysłany: Wto 11 Lip, 2006 |
|
|
KnofeL, to juz jest poprawione w pierwszym poscie? |
|
KnofeL Wysłany: Wto 11 Lip, 2006 |
|
|
Ok. Jest nowsza werjs niestety dalej bez autoryzacji :( (Ale cierpliswosci)
Zmiany:
- Wszystko to co zauwa?y? PiotrLegnica - Za co jeszcze raz bardzo dzi?kuj? ;)
- Jeden u?ytkownik mo?e wys?ac tylko jedno zg?oszenie (Wcze?niej kiedy raz co? wys?a?, potem wys?ala jeszcze raz z innymi danymi to dane te by?y podmieniane)
- Poprawione wykrywanie niewype?nionych p?l.
- Inne drobne zmiany kodu.
- Ulatwienie instalacji.
Co planuje
- Zrobinie w koncu tej nieszcz?snej autoryzacji
- Mo?liwos? wybrania czy pole jest obiw?zkowe czy nie.
To narazie tyle. Sorki za op??nienie |
|
shout_ Wysłany: Sob 15 Lip, 2006 |
|
|
A czy kto? si? mnie spyta? czy wyra?am zgod? :? ? Wko?cu to by? m?j pomys?... |
|
PiotrLegnica Wysłany: Sob 15 Lip, 2006 |
|
|
shout_ napisał/aWko?cu to by? m?j pomys?...
Pisze si? "w ko?cu" to raz, a dwa, to masz na to jaki? patent? Ten mod nie korzysta ani nie bazuje na twoim, wi?c nikt nie musi ci? pyta? czy si? zgadzasz czy nie. |
|
shout_ Wysłany: Sob 15 Lip, 2006 |
|
|
CytatPisze si? w ko?cu to raz, a dwa, to masz na to jaki? patent? Ten mod nie korzysta ani nie bazuje na twoim, wi?c nikt nie musi ci? pyta? czy si? zgadzasz czy nie.
Bazuje na moim, bo ja mia?em taki pomys?, a pisa?o "Copyright !" Nie mam nic przeciwko, ale warto si? zapyta? o zgode autora :/. |
|
|