search

PHP-Fusion-GERMANY / PHP-Fusion

Vom deutschen Team weiterentwickelter Fork
http://phpfusion-deutschland.de
6 stars 3 forks source link

Update setup.php #7

Closed Webmeteor closed 7 years ago

Webmeteor commented 8 years ago

add utf8 support to tables, add meta description and meta keywords to tables

iPaat commented 8 years ago

Hallo, vielen Dank für deine Einsendung.

Leider kann ich dies so (noch) nicht annehmen. Was ich nicht verstehe ist, wofür die Meta Daten sind? Könntest du dich in diesem PR bitte nur auf den UTF8 Support beschränken?

Webmeteor commented 8 years ago

Fürs onpage SEO. Bisher gibt es das ja nur global, also jede Seite hat die gleiche meta description und die gleichen meta keywords.

iPaat commented 8 years ago

Ich würde das wie gesagt eventuell später nachholen da wir nicht einfach so das Datenbank Schema ändern dürfen. Wir müssen auch an Updates denken usw.

iPaat commented 8 years ago

Hallo, vielen Dank.

Ich habe den PR erst einmal als zurückgestellt markiert bis die allgemeine Marschrichtung bekannt ist.

iPaat commented 8 years ago

Ich würde dich bitten deine Branch mit develop zu mergen und dann nochmal einen Commit zu machen. Danach werde ich dann deine Änderungen übernehmen, da ich denke dass wir um PDO nicht herum kommen werden.

Bitte zusätzlich, vielleicht hier in den Kommentaren oder bei #18 , ein Script bereitstellen welches die Datenbank für bestehende Systeme aktualisiert.

Webmeteor commented 7 years ago

see new pull request