| Autor | Zpráva | ||
|---|---|---|---|
| Jan Sehnoutka Profil * |
#1 · Zasláno: 6. 10. 2010, 19:21:34
Vytvořil jsem jednoduchý redakční systém pro správu článků. Nejsem programátor a tak jsem nehleděl na čistotu zdrojového kódu, ale na funkčnost. V budoucnu chci tento RS rozšířit na plnohodnotný redakční systém ke správě celého webu.
Uvítám vaše názory, postřehy, nebo i spolupráci. Více info na http://www.articlers.jansehnoutka.cz |
||
| johnl Profil |
#2 · Zasláno: 6. 10. 2010, 19:49:08
Demo nefunguje, a stahovat se mi to nechce.. Ale jinak ten web je hodně jednoduchej a nemáš tam měnící se title, ale špatný to není.. Hodnotil bych tak 8/10 ;-)
|
||
| Jan Sehnoutka Profil * |
#3 · Zasláno: 6. 10. 2010, 20:18:06
johnl:
Omlouvám se, demo už běží ;) |
||
| Hotdreams Profil |
#4 · Zasláno: 6. 10. 2010, 20:32:16
Jde to odinstalovat? Zakaž odinstalaci kdyžtak pro demo.
Napsal jsem ti na icq. |
||
| panther Profil |
#5 · Zasláno: 6. 10. 2010, 20:38:51
Jan Sehnoutka:
„Omlouvám se, demo už běží ;)“ opravdu? Table '4341-articlers.ars_system' doesn't exist. Ať tak či tak, proč tyhle chybové hlášky vypisuješ? K uživateli by se vůbec neměly dostat. Mno, když to tak vidím, asi bych si u tebe nic nekoupil. Je to dost o strach o svá data. |
||
| Jan Sehnoutka Profil * |
#6 · Zasláno: 6. 10. 2010, 20:42:53
panther:
Chyba byla v tom že jsem zapomněl zablokovat odinstalování systému v demo verzi. Ted už to pojede. Jinak se omlouvám. |
||
| colorshock Profil |
#7 · Zasláno: 6. 10. 2010, 20:57:53
Administrace hezka.. ale to publikovani článků by chtělo alespoň nějakou základní šablonu.
|
||
| Jan Sehnoutka Profil * |
#8 · Zasláno: 6. 10. 2010, 20:59:15
colorshock:
Myslíš nastylovat CSSkem nějakou výchozí šablonu na zobrazování článků? |
||
| Keeehi Profil |
#9 · Zasláno: 6. 10. 2010, 21:05:03 · Upravil/a: Keeehi
Jan Sehnoutka:
Na pohled (graficky) to vypadá příjemně, instalovat jsem to nezkoušel, ale ten postup instalace vypadá lehce. Na stránkách mě zarazila doporučená konfigurace. 5.2 nebo vyšší. Prolítl jsem zdroják a nikde nenašel nějaké nové funkce. Naopak používáš EregI_Replace(); která je dnes už deprecated. Další věcí je, že zapisuješ pole jako $cfg[host], správně je to však $cfg['host']
Ve složce popups a styles máš jeden stejný soubor vždy 2x. Netuším proč. Nevím, k čemu potřebuješ soubor robots.txt s takovýmto obsahem, ale jeho umístění není správně. Měl by být o složku výše. Takhle je tam nefunkční. Neber to jako nějakou výraznou kritiku, spíš jako postřehy k vylepšení. WYSIWYG editor nefunguje v Google chrome 7.0.536.2 dev (vyskočí alert, že není podporován) - můžeš ho zkusit aktualizovat. |
||
| Jan Sehnoutka Profil * |
#10 · Zasláno: 6. 10. 2010, 21:18:18
Keeehi:
Díky za připomínky. Budu to dál vylepšovat. |
||
| colorshock Profil |
#11 · Zasláno: 6. 10. 2010, 22:06:34
jj nastylovat.. preci jen to bude vypadat na oko lepe a zaujme to..
|
||
| Jan Sehnoutka Profil * |
#12 · Zasláno: 7. 10. 2010, 13:40:08
colorshock:
Už se na tom pracuje ;) |
||
| Jan Sehnoutka Profil * |
#13 · Zasláno: 9. 10. 2010, 12:16:36
Vydal jsem další verzi, ve které jsou opraveny věci, ke kterým jste měli připomínky + nový WYSIWYG editor.
|
||
| Keeehi Profil |
#14 · Zasláno: 9. 10. 2010, 14:49:35
ještě jsem našel
$spojeni = mysql_connect("$cfg[host]", "$cfg[login]", "$cfg[password]");Pořád se tam vyskytují funkce EregI_Replace. comment-post.php admin/comment-post.php admin/add-article.php To echo začínající na 49. řádku by se dalo zapsat lépe. <?php if (true) ?>
moc dlouhý
html kód
který je zbytečný
vypisovat echem
<?php }
else {
?>
else větev
<?php }?>admin/edit-user.php if - else je tu špatně uděláno. Ten kód je téměř totožný. To co je stejné by mělo zůstat statické a doplňovat jenom to, co se mění. admin/lost-pas-form.php Je statický soubor, proč tam používáte PHP? admin/show-article.php Zase se tu vyskytují větší statické bloky html vypisované echem. ř. 232-239 by sel krásně zapsat cyklem. Soubor phpinfo.php by se tam neměl vůbec vyskytovat! Ten kdo bude tento RS používat ho vůbec nepotřebuje. Řekl bych, že je tu možnost, aby se útočník dostal do databáze. Vaše demo před útokem naštěstí chrání nastavení php. Ovšem ti co si Váš RS nainstalují, už takové štěstí mít nemusí. Proto MUSÍTE kontrolovat VŠECHNY vstupy, z kterých se vytváří později SQL dotaz! |
||
| Lamicz Profil |
#15 · Zasláno: 9. 10. 2010, 21:41:44
escapovat, escapovat, escapovat...
|
||
| dRaGen Profil |
#16 · Zasláno: 9. 10. 2010, 23:24:21
A přesně takhle to dopadá, když se NEprogramátor snaží napsat redakční systém... říká ti něco třeba MVC/P návrh? OOP?
|
||
| Mastodont Profil |
#17 · Zasláno: 10. 10. 2010, 08:11:24
dRaGen:
Nikdo učený z nebe nespadl. |
||
| Jan Sehnoutka Profil * |
#18 · Zasláno: 12. 10. 2010, 18:51:57
Keeehi:
Díky za upozornění na chyby, přesně to potřebuji ;) |
||
|
Časová prodleva: 3 dny
|
|||
| Hotdreams Profil |
#19 · Zasláno: 15. 10. 2010, 12:31:10
dRaGen:
myslím s iže uživatelovi nevadí že to nepoužívá, jediné co chce je správu s články.. |
||
| F1eldy Profil |
#20 · Zasláno: 15. 10. 2010, 16:08:33
Hotdreams:
Přesně tak, hlavní je aby vše fungovalo, čistý kód je samozřejmě pěkná věc, ale uživatele bude nejvíce zajímat funkčnost. |
||
|
Časová prodleva: 17 dní
|
|||
| F1eldy Profil |
#21 · Zasláno: 1. 11. 2010, 14:54:30
Nově založeno fórum. Veškeré problémy, dotazy a nápady řešte tam. http://www.articlers.jansehnoutka.cz/forum/
|
||
|
Časová prodleva: 13 let
|
|||
0