Framework: A migração não leva em conta o prefixo ao verificar se a tabela de migração existe [bug]

Criado em 11 jan. 2013  ·  3Comentários  ·  Fonte: laravel/framework

Bug original encontrado aqui: https://github.com/illuminate/database/issues/111 - Movido para seu repositório de acordo com Taylor. Aqui está o texto original:

Falei com Machuga no IRC - Foi sugerido que eu criasse um problema.

Emitir:

Erro após a primeira migração: SQLSTATE[42S01]: Base table or view already exists: 1050 Table 'up_migrations' already exists

Passos para reproduzir:

  1. Nova instalação de L4
  2. Adicionar um prefixo ao banco de dados na configuração de conexão do banco de dados (MySql)
  3. Crie uma migração $ php artisan migrate:make create_users_table --table=users --create
  4. Preencha alguns campos, execute a migração $ php artisan migrate
  5. Tente uma migração: atualize $ php artisan migrate:refresh
  6. ERRO: SQLSTATE[42S01]: Base table or view already exists: 1050 Table 'up_migrations' already exists

    Arquivos Relevantes:

Eu rastreei este arquivo : Illuminate\Database\MigrationsDatabaseMigrationRepository::repositoryExists() e especificamente dentro dele, a chamada para return $schema->hasTable($this->table); aqui

A variável $ this-> table passada para hasTable () não inclui o prefixo da tabela. Illuminate\Database\Schema\MySqlBuilder::hasTable($table) também não verifica o prefixo.

Infelizmente, ainda não estou familiarizado com o código / convenção para saber onde você prefere procurar o prefixo. (Não tenho certeza de que classe deve ter esse "conhecimento")

bug

Todos 3 comentários

OK, obrigado. Nós vamos consertar isso.

Fixo.

Estou tendo esse mesmo problema e acabei de baixar o framework do site.
Eu me pergunto se a correção foi comprometida com a versão do site.

Esta página foi útil?
0 / 5 - 0 avaliações

Questões relacionadas

SachinAgarwal1337 picture SachinAgarwal1337  ·  3Comentários

CupOfTea696 picture CupOfTea696  ·  3Comentários

ghost picture ghost  ·  3Comentários

Fuzzyma picture Fuzzyma  ·  3Comentários

iivanov2 picture iivanov2  ·  3Comentários