[PR #9] [MERGED] Bug fixes 2012/10 #44

New Issue

admin · 2026-01-23T11:35:01+01:00

admin commented

2026-01-23 11:35:01 +01:00

📋 Pull Request Information

Original PR: https://github.com/doctrine/KeyValueStore/pull/9
Author: @jonmchan
Created: 10/3/2012
Status: ✅ Merged
Merged: 11/19/2012
Merged by: @beberlei

Base: master ← Head: bugFixes-2012-10

📝 Commits (7)

aea2dad updated bootstrap path to comply with new path (see https://github.com/composer/composer/issues/497)
066b047 updated ClassMetadataFactory to fit doctrine (09cf5f63bc) changes in AbstractClassMetadataFactory
46f5825 updated ClassMetadataFactory to fit doctrine (c6a01cc34e) changes in AbstractClassMetadataFactory
479e2cf updated readme to reflect changes in EntityManager constructor - fixes #8
2068743 updated readme with CRUD examples
e4e5e14 KeyValueStore does not support mapped superclass - any mapped class should be considered as an entity.
c5b8c19 corrected erroneous readme

📊 Changes

3 files changed (+34 additions, -5 deletions)

View changed files

📝 README.md (+24 -3)
📝 lib/Doctrine/KeyValueStore/Mapping/ClassMetadataFactory.php (+9 -1)
📝 tests/bootstrap.php (+1 -1)

📄 Description

Here's the changes that I had to fix to get it working with the latest dev version of doctrine/common. I also updated the README to reflect code changes and to add clarification on how to use the EntityManager.

_{🔄 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/KeyValueStore/pull/9 **Author:** [@jonmchan](https://github.com/jonmchan) **Created:** 10/3/2012 **Status:** ✅ Merged **Merged:** 11/19/2012 **Merged by:** [@beberlei](https://github.com/beberlei) **Base:** `master` ← **Head:** `bugFixes-2012-10` --- ### 📝 Commits (7) - [`aea2dad`](https://github.com/doctrine/KeyValueStore/commit/aea2dadbc4d5c7d8dc2f58b00dacbfcec8b6ef2c) updated bootstrap path to comply with new path (see https://github.com/composer/composer/issues/497) - [`066b047`](https://github.com/doctrine/KeyValueStore/commit/066b047c13cbfb1dcce2fbd582d674d790c34c63) updated ClassMetadataFactory to fit doctrine (https://github.com/doctrine/common/commit/09cf5f63bc0276f360bf40c9086d9477f31edac5) changes in AbstractClassMetadataFactory - [`46f5825`](https://github.com/doctrine/KeyValueStore/commit/46f58250de950bdf7077c35e7bdc3d1be956dbac) updated ClassMetadataFactory to fit doctrine (https://github.com/doctrine/common/commit/c6a01cc34ec3907a797cd89a3af56e07f4fee66e) changes in AbstractClassMetadataFactory - [`479e2cf`](https://github.com/doctrine/KeyValueStore/commit/479e2cf739e5d56a8c6248f2c541808b372e9e0d) updated readme to reflect changes in EntityManager constructor - fixes #8 - [`2068743`](https://github.com/doctrine/KeyValueStore/commit/20687435bdac9823af001e13a41f164df7235929) updated readme with CRUD examples - [`e4e5e14`](https://github.com/doctrine/KeyValueStore/commit/e4e5e14a797ec666f8650dc24b6556902e27ff24) KeyValueStore does not support mapped superclass - any mapped class should be considered as an entity. - [`c5b8c19`](https://github.com/doctrine/KeyValueStore/commit/c5b8c1931d139d4a97d340e0ca0ce3ecffbcf680) corrected erroneous readme ### 📊 Changes **3 files changed** (+34 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+24 -3) 📝 `lib/Doctrine/KeyValueStore/Mapping/ClassMetadataFactory.php` (+9 -1) 📝 `tests/bootstrap.php` (+1 -1) </details> ### 📄 Description Here's the changes that I had to fix to get it working with the latest dev version of doctrine/common. I also updated the README to reflect code changes and to add clarification on how to use the EntityManager. --- <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-23 11:35:01 +01:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: doctrine/KeyValueStore#44