ademcan / canSnippet

An open source web-based snippets management tool
MIT License
48 stars 7 forks source link

Correction de quelque bug mais surtout des caractére spéciaux PHP>=5.4 #1

Closed bawaaaaah closed 10 years ago

bawaaaaah commented 10 years ago

La plupart de ses bug apparaissent a partir de la verssion 5.4 de PHP Correction de diverse bug liée au nouveau comportement de PHP 5.4 les htmlentites nécessite maintenant de préciser l'encodage car celui par defaut et UTF-8, ce qui pose probléme avec le site en ISO-8859-15

initialisation de action si la conf PHP n'initialise pas les variable utilisée et inexistante

suppression du langage afficher en haut a gauche dans les modification correction du nom de champ dans dans les modification correction du comportement des checkbox private correction d'un iconv buguer dans les modification le status private et maintenant bien modifier fix du comportemzent des checkbox private

Modification du script d'install pour check les permission lors de la création due la DB

ajout d'instruction dans le readme pour proteger du téléchargement de la bdd sqlite pour nginx ajout d'un htaccess pour apache qui protége les fichier sqlite

baptistedonaux commented 10 years ago

+1 Supprimer snippets.sqlite était nécessaire. As-tu ajouter une option pour quelle soit automatiquement créer ?

bawaaaaah commented 10 years ago

L'install.php est justement la pour générer le sqlite.

baptistedonaux commented 10 years ago

Je demande parce que ce n'est pas le cas aujourd'hui.

bawaaaaah commented 10 years ago

je vien de tester ça marche pour ma part, as tu des erreur? peut tu les remonter. as tu essayer d'ouvrir directement depuis ton serveur la page install.php?

baptistedonaux commented 10 years ago

Cela ne fonctionne ni sur une Debian, ni sur Mac OS.

Fatal error: Uncaught exception 'Exception' with message 'Unable to open database: unable to open database file' in /Applications/XAMPP/xamppfiles/htdocs/canSnippet/install.php:49 Stack trace: #0 /Applications/XAMPP/xamppfiles/htdocs/canSnippet/install.php(49): SQLite3->__construct('snippets.sqlite') #1 {main} thrown in /Applications/XAMPP/xamppfiles/htdocs/canSnippet/install.php on line 49

Je te laisse gérer l'erreur puisque c'est ta PR. Tiens moi au jus pour tester.

bawaaaaah commented 10 years ago

1 petite question encore pour debian, as tu les droit d'ecriture sur ton repertoire avec l'utilisateur www-data sur ton repertoire?

baptistedonaux commented 10 years ago

Bien vu, je viens juste de forker le projet, je n'ai pas changé les droits. Une alerte pour indiquer le soucis pourrait être pas mal je pense.

Qu'en penses-tu ?

bawaaaaah commented 10 years ago

je suis d'accord avec toi je vaios bosser dessus. merci de la remonter

baptistedonaux commented 10 years ago

Pas de quoi ;)

ademcan commented 10 years ago

Merci énormément pour votre aide, de mon côté je suis un peu perdu, c'est bizarre github, il faut vraiment que je m'y habitue. Je vais essayer de faire les choses correctement, ne vous inquiétez pas si vos pull request prennent un peu de temps.

baptistedonaux commented 10 years ago

Pas de soucis, n'hésites pas à demander à ceux qui en propose plus d'informations.

bawaaaaah commented 10 years ago

je vien de finir la modification du script d'install, je gére même les permission si la DB se trouve dans un repertoire a part. et je vien de pensser que le fichier et dispo a tous en libre service (tu tape le lien http://127.0.0.1/snippset.sqlite et pouf)

bawaaaaah commented 10 years ago

thx, PS: faudrait tester le .htaccess j'ai pas de apache sous la main j'ai qu'un NGINX

baptistedonaux commented 10 years ago

Je peux. Je fais ça

bawaaaaah commented 10 years ago

thx

ademcan commented 10 years ago

Les amis ça avance trop vite pour moi... J'ai accepté tous les merge pour des raisons de sécurité, mais je vais prendre mon temps pour la suite. Je ne veux pas perdre le contrôle dès le début :s c'est pas super simple github, j'aimais bien SVN moi :)

bawaaaaah commented 10 years ago

y'a pas de soucis

baptistedonaux commented 10 years ago

Ton .htaccess est ok de mon côté. Pas d'accès ! Bravo

bawaaaaah commented 10 years ago

quelqu'un peut me valider la présence de mon dernier patch aprés le merge qui a était fait? (la présence du .htaccess etcc... dans le projet principals) j'ai l'impression qu'il est pas passer entiérement

baptistedonaux commented 10 years ago

Il est passé. Je l'ai sur le dépôt principal.

bawaaaaah commented 10 years ago

ok, merci