Development

#3476 ([PATCH] Use of class constant in propel:build-forms requires application configuration if model includes plugin behaviors)

You must first sign up to be able to contribute.

Ticket #3476 (reopened defect)

Opened 2 weeks ago

Last modified 3 days ago

[PATCH] Use of class constant in propel:build-forms requires application configuration if model includes plugin behaviors

Reported by: Kris.Wallsmith Assigned to: fabien
Priority: minor Milestone:
Component: generator Version: 1.1.0 DEV
Keywords: Cc:
Qualification: Unreviewed

Description

If your model includes plugin behaviors, the current propel:build-forms task will fail since it does not pull in the plugin config.php files that register those behaviors.

One solution would be to load an application configuration instead of just the project configuration for this task, but that would have repercussions in the current propel:build-all task.

I've attached a patch that catches this particular exception in the forms generator. There may be a more elegant/sophisticated solution out there, but I'll leave that to the core team.

Attachments

propel-build-forms-behaviors.patch (0.9 kB) - added by Kris.Wallsmith on 05/05/08 15:59:31.
propel-build-forms-behaviors-v2.patch (3.2 kB) - added by Kris.Wallsmith on 05/15/08 01:16:56.
updated patch to rollback earlier patch

Change History

05/05/08 15:59:31 changed by Kris.Wallsmith

  • attachment propel-build-forms-behaviors.patch added.

05/09/08 14:40:51 changed by Kris.Wallsmith

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

This represents the same bug as #3502, which fabien just added to the 1.1.0 RC2 milestone.

05/10/08 14:49:07 changed by fabien

  • resolution changed from duplicate to fixed.

(In [8896]) fixed behavior registration bug in propel tasks (closes #3502, #3476, #3470 - patch from Kris.Wallsmith)

05/15/08 01:15:32 changed by Kris.Wallsmith

  • status changed from closed to reopened.
  • resolution deleted.

#3543 reports this bug has not been resolved. I've attached a patch that updates the OM and Peer builders to wrap the inclusion of the behaviors file with a conditional that tests for an application configuration.

05/15/08 01:16:56 changed by Kris.Wallsmith

  • attachment propel-build-forms-behaviors-v2.patch added.

updated patch to rollback earlier patch