Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

fix ticket 3429 #998

Merged
merged 1 commit into from

3 participants

@dereuromark
Collaborator

see http://cakephp.lighthouseapp.com/projects/42648/tickets/3429-baked-schema-has-an-excess-semicolon

i still dont like how generateTable() creates whitespace noice.
but this fixes the issue anyway.

@zuborawka

Thank you.

@markstory markstory merged commit 5ccba46 into cakephp:2.3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 3, 2012
  1. @dereuromark

    fix ticket 3429

    dereuromark authored
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 2 deletions.
  1. +2 −2 lib/Cake/Console/Command/Task/FixtureTask.php
View
4 lib/Cake/Console/Command/Task/FixtureTask.php
@@ -279,8 +279,8 @@ public function getPath() {
* @return string fields definitions
*/
protected function _generateSchema($tableInfo) {
- $schema = $this->_Schema->generateTable('f', $tableInfo);
- return substr($schema, 13, -2);
+ $schema = trim($this->_Schema->generateTable('f', $tableInfo), "\n");
+ return substr($schema, 13, -1);
}
/**
Something went wrong with that request. Please try again.