Skip to content

Commit

Permalink
Fix missing comment field in c_calendar_event
Browse files Browse the repository at this point in the history
  • Loading branch information
@ywarnier
ywarnier committed May 25, 2015
1 parent f7082f8 commit ded881a
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions main/install/database.sql
View file
Expand Up @@ -3690,6 +3690,7 @@ CREATE TABLE c_calendar_event(
parent_event_id INT NULL,
session_id int unsigned NOT NULL default 0,
all_day INT NOT NULL DEFAULT 0,
comment TEXT,
PRIMARY KEY (id, c_id)
);

Expand Down

7 comments on commit ded881a

@jmontoyaa
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

El cambio se debe realizar en el archivo y no en database.sql

Entity/CCalendarEvent.php

Lo acabo de agregar aqui:

6cc7390

@ywarnier
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Enterado, gracias.

@ywarnier
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the change should not be made in database.sql anymore, is there a reason to maintain database.sql? (it's confusing and it's destined to be generate unsynchronized database scripts...)

@jmontoyaa
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Solamente como guía. Para comprobar si no existe una diferencia con la BD creada por las entidades. Existe una nota en database.sql donde dice no modificar ese archivo.

@ywarnier
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pero data.sql sí se puede modificar, verdad?

@jmontoyaa
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Exacto. data.sql si es usado.

@ywarnier
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK

Please sign in to comment.