[PR #680] [CLOSED] [wip] better reverse engineers the mapping metadata from a database #8548

Closed
opened 2026-01-22 16:00:25 +01:00 by admin · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/doctrine/orm/pull/680
Author: @bronze1man
Created: 5/29/2013
Status: Closed

Base: masterHead: PR-mapping-import-master


📝 Commits (1)

  • 6b347d1 database driver add index comment default

📊 Changes

2 files changed (+78 additions, -1 deletions)

View changed files

📝 lib/Doctrine/ORM/Mapping/Driver/DatabaseDriver.php (+26 -1)
📝 tests/Doctrine/Tests/ORM/Functional/DatabaseDriverTest.php (+52 -0)

📄 Description

The problem:

1.use symfony2 doctrine:mapping:import from msyql database to annotation
2.use symfony2 doctrine:schema:update will see a lot of sql update about just import database schema.
3.this PR fix eliminate some of those sql update...

has fixed problem:

  • column default
  • column unsigned (add test)
  • column comment
  • column type decimal with precision and scale
  • table with simple index
  • table with unique index

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/doctrine/orm/pull/680 **Author:** [@bronze1man](https://github.com/bronze1man) **Created:** 5/29/2013 **Status:** ❌ Closed **Base:** `master` ← **Head:** `PR-mapping-import-master` --- ### 📝 Commits (1) - [`6b347d1`](https://github.com/doctrine/orm/commit/6b347d1755a5a289d77c57c96eec8c09dd123d53) database driver add index comment default ### 📊 Changes **2 files changed** (+78 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `lib/Doctrine/ORM/Mapping/Driver/DatabaseDriver.php` (+26 -1) 📝 `tests/Doctrine/Tests/ORM/Functional/DatabaseDriverTest.php` (+52 -0) </details> ### 📄 Description # The problem: 1.use symfony2 doctrine:mapping:import from msyql database to annotation 2.use symfony2 doctrine:schema:update will see a lot of sql update about just import database schema. 3.this PR fix eliminate some of those sql update... # has fixed problem: - column default - column unsigned (add test) - column comment - column type decimal with precision and scale - table with simple index - table with unique index --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
admin added the pull-request label 2026-01-22 16:00:25 +01:00
admin closed this issue 2026-01-22 16:00:26 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#8548