6.0.0-beta1
7/5/25

[#10443] Wrong autoincrementKey usage
Summary Wrong autoincrementKey usage
Queue Ansel
Queue Version Git master
Type Bug
State Resolved
Priority 2. Medium
Owners mrubinsk (at) horde (dot) org
Requester jan (at) horde (dot) org
Created 08/19/2011 (5069 days ago)
Due
Updated 09/02/2011 (5055 days ago)
Assigned
Resolved 09/02/2011 (5055 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
09/02/2011 04:17:47 PM Michael Rubinsky State ⇒ Resolved
 
09/02/2011 04:17:07 PM Git Commit Comment #4 Reply to this comment
Changes have been made in Git for this ticket:

Bug: 10443 fix another erroneous autoincrementKey change.
Also, 3 and 7 migration were essentially duplicates (7 was to fix an 
error in the 3
migration that has since been fixed in 3). make the 7 migration a noop.

  2 files changed, 9 insertions(+), 88 deletions(-)
http://git.horde.org/horde-git/-/commit/9df7438cc5e57e34d327b498bd98ed3674b58652
09/02/2011 04:17:02 PM Git Commit Comment #3 Reply to this comment
Changes have been made in Git for this ticket:

Bug: 10443 - fix erroneous primaryKey -> autoincrementKey change
Tables seem to have be created correctly with the incorrect usage in
MySQL and Postgres 9, plus all existing content tests were passing. This
just fixes the migrations for new installs.

  1 files changed, 7 insertions(+), 3 deletions(-)
http://git.horde.org/horde-git/-/commit/a233e45407e62132de5653e14146697b54ec5590
08/19/2011 05:28:50 PM Jan Schneider Comment #2
Summary ⇒ Wrong autoincrementKey usage
Reply to this comment
There are more wrong usages, in content:
rampage_tagged and rampage_user_tag_stats use it for multi-column keys
rampage_tag_stats is using it for tag_id, which is never 
autoincremented, but always inserted manually.
08/19/2011 05:12:04 PM Jan Schneider Comment #1
Priority ⇒ 2. Medium
Patch ⇒ No
Milestone ⇒
Assigned to Michael Rubinsky
Queue ⇒ Ansel
Summary ⇒ Wrong column type for style_hash
Type ⇒ Bug
State ⇒ Assigned
Reply to this comment
ansel_hashes.style_hash is defined as an autoincrementKey and string 
column at the same time, which are incompatible.

Saved Queries