Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Algumas alterações no datasource e no .htaccess #1

Open
wants to merge 2 commits into from

2 participants

mikemaciel Eliel de Paula
mikemaciel
Collaborator
  • Incluí no arquivo sys/core/datasource.php uma verificação no método construtor, verificando se o desenvolvedor definiu a variável de ambiente ENVIRONMENT. Também alterei o nome dela na classe, que estava com grafia incorreta;
  • Inclusão de algumas configurações no .htaccess;
  • O arquivo em sys/helpers/sessaoHelper.php foi renomeado, atendendo o padrão adotado, onde são utilizados nomenclaturas em inglês;
mikemaciel added some commits
mikemaciel mikemaciel Verificação se a variável de ambiente foi definida, algumas nomenclat…
…uras padronizadas, adição de instruções no arquivo .htaccess

- Incluí no arquivo sys/core/datasource.php uma verificação no método
construtor, verificando se o desenvolvedor definiu a variável de
ambiente ENVIRONMENT. Também alterei o nome dela na classe, que estava
com grafia incorreta;
- Inclusão de algumas configurações no .htaccess;
- O arquivo em sys/helpers/sessaoHelper.php foi renomeado, atendendo o
padrão adotado, onde são utilizados nomenclaturas em inglês;
37df9eb
mikemaciel mikemaciel Correção no index.php
- Correção do nome da constante que define o ambiente de execução;
- Correção do nome do diretorio do sistema Mojo*PHP;
ffe16d9
Eliel de Paula

.htaccess -> Legal!
index.php (enviroment) valeu, o sono faz coisas na cabeça da gente.
index.php (DIR_INSTALACAO) na verdade se refere ao diretório que o FW está, no caso de não estar na raiz, mas só influencia no BASE_URL pois o BASE_PATH pega o caminho completo, enquanto que o $_SERVER só pega o host, se estiver em uma pasta já não referencia direito.

Ainda não peguei nenhum problema com isso no desenvolvimento do site oficial, mas já passei por isso em projetos anteriores, mas derrepente a gente acha uma solução mais elegante e elimina essa constante.

Collaborator

Não investiguei a fundo, mas depois que eu alterei o valor de DIR_INSTALACAO para "sys" (antes estava "MojoPHP") um erro louco de 404 not found que eu estava tendo na requisição do /index.php sumiu. Development Environment in MAMP. Confere o histórico aí:

31 -define('DIR_INSTALACAO', 'MojoPHP'); <-- ESTAVA ASSIM
30 +define('DIR_INSTALACAO', 'sys'); <-- FICOU ASISM

Opa, eu testei o Mojo*PHP on-line em www.elieldepaula.com.br/mojo e essa variável realmente fez a diferença na hora de exibir o layout da view. TAlvez seja questão de renomear para uma coisa mais intuitiva.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 2 unique commits by 1 author.

Mar 01, 2012
mikemaciel mikemaciel Verificação se a variável de ambiente foi definida, algumas nomenclat…
…uras padronizadas, adição de instruções no arquivo .htaccess

- Incluí no arquivo sys/core/datasource.php uma verificação no método
construtor, verificando se o desenvolvedor definiu a variável de
ambiente ENVIRONMENT. Também alterei o nome dela na classe, que estava
com grafia incorreta;
- Inclusão de algumas configurações no .htaccess;
- O arquivo em sys/helpers/sessaoHelper.php foi renomeado, atendendo o
padrão adotado, onde são utilizados nomenclaturas em inglês;
37df9eb
mikemaciel mikemaciel Correção no index.php
- Correção do nome da constante que define o ambiente de execução;
- Correção do nome do diretorio do sistema Mojo*PHP;
ffe16d9
This page is out of date. Refresh to see the latest.
21 .htaccess
... ... @@ -0,0 +1,21 @@
  1 +SetEnv ENVIRONMENT desenvolvimento
  2 +
  3 +<Files ~ "^.(htaccess|htpasswd)$">
  4 + deny from all
  5 +</Files>
  6 +
  7 +<IfModule mod_rewrite.c>
  8 +#Options +FollowSymLinks
  9 +RewriteEngine On
  10 +
  11 +##Forca o uso do www na url
  12 +#RewriteCond %{HTTP_HOST} ^dominio.com.br [NC]
  13 +#RewriteRule ^(.*)$ http://www.dominio.com.br/$1 [L,R=301]
  14 +
  15 +### Inclui exececao de verificacao do www para subdominios
  16 +#RewriteCond %{HTTP_HOST} !^subdominio.dominio.com.br [NC]
  17 +
  18 +RewriteBase /
  19 +RewriteCond %{REQUEST_URI} !^(/index\.php|/template|/assets|/admin|/data|/tmp|/robots\.txt|/favicon\.ico)
  20 +RewriteRule ^(.*)$ /index.php/$1 [L]
  21 +</IfModule>
9 index.php 100644 → 100755
... ... @@ -1,12 +1,11 @@
1 1 <?php
2   -
3 2 /**
4 3 * Ambiente da instala��o.
5 4 */
6   -define('ENVIROMENT', 'desenvolvimento');
  5 +define('ENVIRONMENT', 'desenvolvimento');
7 6
8   -if (defined('ENVIROMENT')) {
9   - switch (ENVIROMENT) {
  7 +if (defined('ENVIRONMENT')) {
  8 + switch (ENVIRONMENT) {
10 9 case 'desenvolvimento':
11 10 error_reporting(E_ALL);
12 11 break;
@@ -28,7 +27,7 @@
28 27 /**
29 28 * Diret�rio de instala��o.
30 29 */
31   -define('DIR_INSTALACAO', 'MojoPHP');
  30 +define('DIR_INSTALACAO', 'sys');
32 31 /**
33 32 * Caminho f�sico completo da instala��o.
34 33 */
9 sys/core/datasource.php
@@ -16,8 +16,15 @@ class MJ_Datasource extends MJ_Object {
16 16
17 17 function __construct() {
18 18 parent::__construct();
  19 +
  20 +
  21 + if(!defined(ENVIRONMENT)) {
  22 + throw new Exception('ENVIRONMENT is not set. See your .htaccess file.');
  23 +
  24 + }
  25 +
19 26 $config = Config::read('database');
20   - $driver = $config[ENVIROMENT]['driver'];
  27 + $driver = $config[ENVIRONMENT]['driver'];
21 28 $this->load->driver($driver, NULL, $config);
22 29 $this->db = $this->$driver->getInstance(Config::read('database'));
23 30
0  sys/helpers/sessaoHelper.php → sys/helpers/sessionHelper.php
File renamed without changes

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.