Development

#3473 ([PATCH] schema.yml support for propel's inheritance model)

You must first sign up to be able to contribute.

Ticket #3473 (closed enhancement: fixed)

Opened 5 months ago

Last modified 4 months ago

[PATCH] schema.yml support for propel's inheritance model

Reported by: Kris.Wallsmith Assigned to: dwhittle
Priority: minor Milestone:
Component: model Version: 1.1.0 DEV
Keywords: Cc:
Qualification: Accepted

Description

This patch to the 1.1 branch adds support for Propel's inheritance model to both the old and new schema.yml formats.

Attachments

propel-inheritance.patch (6.0 kB) - added by Kris.Wallsmith on 05/05/08 01:09:33.
propel-inheritance-patch2.patch (3.1 kB) - added by FabianLange on 05/15/08 08:45:06.
need to apply patch 2 on top of patch 1 refs #3545

Change History

05/05/08 01:09:33 changed by Kris.Wallsmith

  • attachment propel-inheritance.patch added.

05/05/08 08:57:35 changed by fabien

  • milestone set to 1.1.0 RC1.

(follow-up: ↓ 3 ) 05/05/08 09:05:12 changed by FabianLange

  • owner changed from fabien to dwhittle.
  • qualification changed from Unreviewed to Accepted.

as far as I know this is propel 1.3 feature only, am I wrong? In that case I propose that dustin adds it to his propel 1.3 version of the plugin.

I check the patch and it looks ok, would be perhaps handy to have it in.

(in reply to: ↑ 2 ) 05/05/08 09:24:40 changed by fabien

  • owner changed from dwhittle to fabien.
  • component changed from plugins to model.

No, inheritance is supported by Propel 1.2

05/06/08 15:33:27 changed by FabianLange

  • owner changed from fabien to FabianLange.
  • status changed from new to assigned.

05/06/08 20:01:00 changed by FabianLange

  • owner changed from FabianLange to dwhittle.
  • status changed from assigned to new.

applied support to symfony 1.1 propel 1.2 plugin in r8815. assigning to dustin to check if it can be merged to propel 1.3 plugin without issues as the 1.3 should support this as well :-)

05/07/08 11:02:27 changed by nicolas

  • milestone deleted.

I remove the 1.1.0 RC1 milestone as this is not a bug, but I leave it open anyway for Dustin :)

05/15/08 08:45:06 changed by FabianLange

  • attachment propel-inheritance-patch2.patch added.

need to apply patch 2 on top of patch 1 refs #3545

06/11/08 23:31:17 changed by dwhittle

  • status changed from new to closed.
  • resolution set to fixed.

Can you update this patch against my branch (from which I merged the inheritance support in 1.1 branch). The inheritance support should already be there according to the patches I have merged. I will close for now as I think the changes are there. If not, please update the patch and reopen.