Forum PHP.pl > [projekt] cms autorski

Pomoc - Szukaj - Użytkownicy - Kalendarz

kayman

23.07.2012, 19:40:56

popełniłem cms, z zalozenia mial byc podstawa to wiekszych projektow typu katalog lub serwis ogloszeniowy a co wyszlo ocencie sami

- cms jest oparty o baze mysql
- skorki wykonywane sa arteesteer 3.1 wiec by uzyskac nowy wyglad wystarczy wymienic grafiki i css a przy innym ukladzie strony wystarczy przestawic kilka elemetow
- obecnie poza tym ze jest cms wykonuje automatycznie sietmap i zarzadza .htaccess

skrypt jest odblokowany jednak prosiłbym raczej dodawać niż kasować (chyba ze po sobie)

ostatnio dodałem:

- katalog + możliwość komentowania wpisów
- możliwość komentowania artykułów
- mapę serwisu
- kanały rss
- reklamy
- liczniki, statystyki

http://cms.siezamawia.pl/

dostep do zaplecza - linki sa na stronie (obecnie zablokowałem, w razie chęci obejrzenia zaplecza proszę o kontakt)

wszelkie uwagi mile widziane

jeżeli nie napisałem w odpowiednim dziale bardzo przepraszam

pozdrawiam

e:/ dwie inne skorki

http://asd.siezamawia.pl/
http://babskiland.pl/

nospor

23.07.2012, 20:00:52

Wejdź na stronę główną. Miłych dwójek

No w dzisiejszysch czasach takie błędy to już są raczej nie dopuszczalne.

Monter08

23.07.2012, 20:01:22

Z zewnątrz może wyglądać rewelacyjnie, ale nie wiem co się dzieję w środku, jak już chcesz dać do oceny to pochwal się kodem a nie wynikiem wyjściowym.

nospor

23.07.2012, 20:02:35

Albo to:
http://cms.siezamawia.pl/meble-kategoria-200.html
"You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1"

Cytat

ale nie wiem co się dzieję w środku, jak już chcesz dać do oceny to pochwal się kodem a nie wynikiem wyjściowym.

Po takich babolach widać co jest wewnątrz....

kayman

23.07.2012, 20:03:09

zatem wylacze zaplecze

pozdr

Monter08

23.07.2012, 20:04:27

Cytat(kayman @ 23.07.2012, 21:03:09 )

zatem wylacze zaplecze

pozdr

Tylko, że XSS został osadzony w komentarzach

(teraz to naprawdę boję się co tam jest w środku

)

nospor

23.07.2012, 20:10:42

A link do kategorii też podałem z palca. Do zaplecza nawet nie zaglądałem

http://cms.siezamawia.pl/meble-kategoriaaaaaaa-2.html
I dostaję domyślny 404 apache. Czyli rozumiem, ze ty te wszystkie linki kategorii trzymasz w .htaccess??

kayman

23.07.2012, 20:11:34

temat do kosza

rozumiem ze bubel

pozdrawiam wszystkich komentujących

nospor

23.07.2012, 20:14:37

Nie tyle bubel, co ma parę błędów. Wystarczy je poprawić.

kayman

23.07.2012, 20:26:51

ano trzeba -> na poczatek filtrowanie mocniejsze

Mephistofeles

23.07.2012, 21:42:34

Bez przesady, porównaj to z dziełem Materkamila, a przekonasz się, że tak źle to nie jest

Monter08

23.07.2012, 22:10:03

człowiek uczy się na błędach, ale dobrze, że napisałeś o swoim projekcie, przynajmniej wiesz przy czym musisz więcej posiedzieć i nie będziesz tych błędów popełniał na przyszłość.

kayman

23.07.2012, 22:35:49

teraz żałuje ze wykasowałem te ataki java ale z tego co przeczytałem to wystarczy strip_tags zastosować (tak myślę bo rozwiazania nikt nie podal)

co do .htaccess to jest do dopracowania

!*!

24.07.2012, 09:29:09

Ataki java? Kot i kotara to dla Ciebie też to samo?

kayman

24.07.2012, 10:36:18

Cytat(!*! @ 24.07.2012, 10:29:09 )

Ataki java? Kot i kotara to dla Ciebie też to samo?

to samo

wybacza ale juz nie miales sie do czego przyczepic

proszę moderatora o zamkniecie tematu

ciekawskiii

24.07.2012, 20:24:40

Kolego Kayman cos mi sie wydaje ze sie przejales komentarzami? niepotrzebnie, mysle ze troche nad tym siedziales wiec pracuj nad tym dalej a takie komentarze i wytykanie bledow bardzo pomaga. Popraw bezpieczenstwo, dopracuj szczegoly itp. i odswiez temat.

Poczytaj wspomniany temat z cms materkamila i zobacz co on stworzyl, nie mam czasu testowac ale odrazu po wejsciu na strone glowna widac ogromna roznice.

Shido

25.07.2012, 07:31:36

Dokładnie nie ma czym się przejmować, sam nie lubię krytyki, ale o ile robią to normalni ludzie ( zdecydowana większość tutaj ) to ta krytyka ma ci pomóc, a nie zgnoić.

Jakoś tak dokładnie nie testowałem, też bez dostępu do panelu admina za bardzo nie można ocenić praktycznej części strony.
Na pewno średnio podoba mi się "standardowa" kolorystyka linków, ale to na pewno jest pod systemem (czy czymś w podobię) skórek.

Cytat(ciekawskiii @ 24.07.2012, 21:24:40 )

Poczytaj wspomniany temat z cms materkamila i zobacz co on stworzyl, nie mam czasu testowac ale odrazu po wejsciu na strone glowna widac ogromna roznice.

Nowy sposób motywowania

kayman

25.07.2012, 12:12:11

skórki sa wikonywane programem artisteer 3.1 -> nie mam zdolnosci atystycznych wiec takie rozwiazanie wydale mi sie sluszne i dalo mi tyle skorek ile moze wypluc ten program

calosc powstala w CS3 wiec funkcja filtrujaca sql jest z niego wzieta

Kod

<?php
if (!function_exists("GetSQLValueString")) {
function GetSQLValueString($theValue, $theType, $theDefinedValue = "", $theNotDefinedValue = "")
{
  $theValue = get_magic_quotes_gpc() ? stripslashes($theValue) : $theValue;

  $theValue = function_exists("mysql_real_escape_string") ? mysql_real_escape_string($theValue) : mysql_escape_string($theValue);

  switch ($theType) {
    case "text":
      $theValue = ($theValue != "") ? "'" . $theValue . "'" : "NULL";
      break;
    case "long":
    case "int":
      $theValue = ($theValue != "") ? intval($theValue) : "NULL";
      break;
    case "double":
      $theValue = ($theValue != "") ? "'" . doubleval($theValue) . "'" : "NULL";
      break;
    case "date":
      $theValue = ($theValue != "") ? "'" . $theValue . "'" : "NULL";
      break;
    case "defined":
      $theValue = ($theValue != "") ? $theDefinedValue : $theNotDefinedValue;
      break;
  }
  return $theValue;
}
}
?>

z niego tez pochodza spry to walidacji pol formularzy

cala skorka po przygotowaniu wyglada tak

Kod

<?php
if (!isset($_SESSION)) session_start();
require_once('php/define.php');
require_once('php/GetSQLValueString.php');
require_once('php/class_headSite.php');
$strona = new headSite;
?><!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"[]>
<html xmlns="http://www.w3.org/1999/xhtml" dir="ltr" lang="pl-PL" xml:lang="pl">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
<?php $strona->get_metatags()?>
<base href="<?php echo $strona->base ?>" />
<title><?php echo $strona->title ?></title>
<link rel="stylesheet" href="<?php echo SITE ?>style.css" type="text/css" media="screen" />


<script type="text/javascript" src="<?php echo SITE ?>jquery.js"></script>
<script type="text/javascript" src="<?php echo SITE ?>script.js"></script>
<?php if ($strona->validation) require_once('php/validation.php'); ?>
<style type="text/css">
.post .layout-item-0 {
    padding-right: 10px;
    padding-left: 10px;
}
.ie7 .post .layout-cell {
    border:none !important;
    padding:0 !important;
}
.ie6 .post .layout-cell {
    border:none !important;
    padding:0 !important;
}
</style>
</head>
<body>
<div id="main">
  <div class="cleared reset-box"></div>
  <div class="bar nav">
    <div class="nav-outer">
      <div class="nav-wrapper">
        <div class="nav-inner">
          <?php $strona->get_menu_gorne()?>
        </div>
      </div>
    </div>
  </div>
  <div class="cleared reset-box"></div>
  <div class="header">
    <div class="header-position">
      <div class="header-wrapper">
        <div class="cleared reset-box"></div>
        <div class="header-inner">
          <div class="headerobject"></div>
          <div class="logo">
            <h1 class="logo-name"><a href="<?php echo SITE ?>">Prosty CMS</a></h1>
            <h2 class="logo-text">strona demonstracyjna</h2>

          </div>
        </div>
      </div>
    </div>
  </div>
  <?php $strona->get_reklama(1);?>
  <div class="cleared reset-box"></div>
  <div class="box sheet">
    <div class="box-body sheet-body">
      <div class="layout-wrapper">
        <div class="content-layout">
          <div class="content-layout-row">
            <div class="layout-cell sidebar1">
              <?php $strona->get_menu_boczne()?>
              <?php $strona->get_reklama(2);?>
              <?php $strona->get_licznik()?>
              <?php $strona->get_statystyki_katalogu()?>
              <div class="cleared"></div>
            </div>
            <div class="layout-cell content">
                <?php $strona->get_reklama(3);?>
                <?php $strona->get_formularz()?>
              <?php $strona->get_articles()?>
              <?php $strona->get_komponent()?>
              <?php $strona->get_cid()?>
              <?php $strona->get_wid()?>
              <?php $strona->get_reklama(4);?>
              <div class="cleared"></div>
            </div>
          </div>
        </div>
      </div>
      <div class="cleared"></div>
      <div class="footer">
        <div class="footer-body"> <?php $strona->rss() ?>
          <div class="footer-text">
            <?php $strona->get_menu_dolne()?>
            <p>Copyright 2012.SieZamawia.pl<br />
             Wszystkie prawa zastrzeżone.</p>
          </div>
          <div class="cleared"></div>
        </div>
      </div>
      <div class="cleared"></div>
    </div>
  </div>
  <div class="cleared"></div>
  <p class="page-footer"></p>
  <div class="cleared"></div>
</div>
</body>
</html>

w wielu przypadkach nie trzeba nic przestawiac by uzyskac nowy wyglad w innym wystarczy przestawic te elementy $strona->get_costam()

pozdrawiam

!*!

25.07.2012, 12:16:28

Typowy kod wygenerowany z automatu, 90% zupełnie nie potrzebne.

kayman

25.07.2012, 13:17:43

Cytat(!*! @ 25.07.2012, 13:16:28 )

Typowy kod wygenerowany z automatu, 90% zupełnie nie potrzebne.

bardzo mozliwe jednak w miare wygodne i niezbyt czasochlonne oraz dajace sporo mozliwosci, na moje do wielu projektow to wystarczy, strony zapleczowe oraz wszelakie wizytowki etc. spokojnie mozna na tym postawic

erix

25.07.2012, 13:23:51

Hmm, ciekawe jak z SEO w przypadku takiego kodu...

--
tak, powyższa treść miała ociekać ironią, jeśli ktoś tego nie zauważył

kayman

25.07.2012, 13:36:24

SEO to bardzo sliski temat,

powiem tak, W3C waliduje (w formularzu do prowadzania wpisu sie przyczepil ale sral go pies), lynx go czyta, SEO bardziej zalezy od tresci, struktury linkow (i anchorow) oraz napiepszania slowami kluczowymi niz od kodu samego w sobie

na pewna sa bledy ale beda sukcesywnie poprawiane

erix

25.07.2012, 13:45:55

Cytat

oraz napiepszania slowami kluczowymi niz od kodu samego w sobie

Bluźnisz. Od semantyki też. A jeśli Twój kod opisuje wyłącznie wygląd, to SEO leży.

FYI, nawet nazwy klas mają znaczenie: http://microformats.org/

Gógiel to umie.

kayman

25.07.2012, 14:03:49

mozesz miec 100% racji tylko wybacz, nie dajmy sie oglupic i robmy swoje, to googlarz korzysta z naszej pracy i robi na tym kase

wiec jak moj tak bedzie class="dupamaryna" to juz jego sprawa jak sobie to przeczyta

erix

25.07.2012, 14:05:28

Ale głupota ze strony osoby tnącej.

Bo klasy opisują wygląd, a nie funkcje. Raz - semantyka (SEO), dwa - dasz klasę pinkbox, klient sobie zażyczy zmiany na zielony, to będziesz po wszystkich szablonach latać i zmieniać? A może - nie daj Boże - zostawisz jak jest? Wtedy życzę powodzenia podczas współpracy w grupie.

kayman

25.07.2012, 14:29:15

nie widze zadnego problemu ze zmiana, wykonanie calej skorki trwa ok 1 godzine w automacie, klasy z artisteer nazywaja sie zawsze tak samo dlatego pozwolilem sobie nawet ten html pociac i wplesc do poszczegolnych funckji i szablonow

te 2 strony sa na tym samym skrypcie (bez katalogu, sam cms) i wymagaly w jedny przypadku wymiany css + obrazki a w drugim przestawienia klilku elementow php

http://asd.siezamawia.pl/
http://babskiland.pl/

jak widzisz nie potrzebuje pracy w grupie, wszystko zrobie sam, a czy dobrze to czas pokaze

!*!

25.07.2012, 15:43:33

Super, tylko że strony z reguły nie bywają jednolite. Z automatu to możesz sobie co najwyżej kawę zrobić, a i tak może trafić Ci się lura.

Cytat

przestawienia klilku elementow php

Od kiedy PHP jest od przesuwania elementów?

kayman

25.07.2012, 15:52:27

Cytat(!*! @ 25.07.2012, 16:43:33 )

Od kiedy PHP jest od przesuwania elementów?

od kiedy ja napisalem cms -> zobacz do kodu ze mozna

erix

26.07.2012, 08:40:07

Cytat

Super, tylko że strony z reguły nie bywają jednolite

Zostaw, dostał tyle argumentów, że się myli; kiedyś trafi na sytuację, w której zmądrzeje, a tak - szkoda naszego czasu.

Cytat

od kiedy ja napisalem cms -> zobacz do kodu ze mozna

Uwierz na słowo, syndrom Alfy i Omegi, prędzej czy później, da Ci w kość... [;

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.