Skip to content

Update scripts/Builder/Components/Model.php #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Aug 10, 2012

Conversation

vguardiola
Copy link
Contributor

Change TAB per spaces

phalcon pushed a commit that referenced this pull request Aug 10, 2012
Update scripts/Builder/Components/Model.php
@phalcon phalcon merged commit d9f897a into phalcon:master Aug 10, 2012
@phalcon
Copy link
Collaborator

phalcon commented Aug 10, 2012

Gracias Victor!

@vguardiola
Copy link
Contributor Author

De nada,
Creo que para el cambio de los FLOAT en la migración se debería hacer
una constante nueva que sea Column::float por que no son iguales las
columnas FLOAT y DECIMAL, he visto que es fácil poner lo en phalcon.c pero
como tengo mi C muy oxidado y ya no he visto donde mas se utiliza os lo
dejo a vosotros :)

También he creado un proyecto para hacer el instalador en Gentoo que es la
distro Linux que utilizo y tiene algunos problemas al compilar pecl
directamente del código: https://bitbucket.org/vguardiola/pecl-phalconphp

Saludos

Victor Guardiola Latorre
http://victor.masquesoft.net
[email protected]
[email protected]
[email protected]
Linux user #288624
TLF:667367484
Skype: vguardiola

or Num#Ext: +(34)931-845517 # 288

http://www.masquesoft.net
http://www.masquejuegos.net
http://www.masquelinux.net
http://www.masquewireless.net

On Fri, Aug 10, 2012 at 9:52 PM, Phalcon [email protected] wrote:

Gracias Victor


Reply to this email directly or view it on GitHubhttps://github.com//pull/6#issuecomment-7654926.

@cybor0 cybor0 mentioned this pull request Sep 21, 2012
@karborator karborator mentioned this pull request Dec 8, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants