Ticket #376 (closed defect: fixed)

Opened 16 years ago

Last modified 14 years ago

generatedViewMapping setting not being appended to viewMappings

Reported by: boomfish Owned by: boomfish
Priority: normal Milestone: 3.1 Maintenance Beta
Version: 3.0.178 Severity: normal
Keywords: Cc:

Description

In Model-Glue 2, The generatedViewMapping config setting was automatically appended to an application's viewMappings. In Model-Glue 3, this is no longer the case.

References:

MG2 source: http://svn.model-glue.com/oldversions/Unity/ModelGlue/unity/framework/ModelGlue.cfc (line 439)

User report: http://groups.google.com/group/model-glue/browse_thread/thread/535a2da0bab80344

Attachments

376.txt (0.8 kB) - added by bob.silverberg@… 16 years ago.
A patch for this bug

Change History

Changed 16 years ago by bob.silverberg@…

A patch for this bug

Changed 16 years ago by bob.silverberg@…

I've attached a patch that fixes this issue.

Changed 16 years ago by DanWilson

  • milestone changed from 3.2 to 3.1.x-next-patch

Changed 16 years ago by boomfish

  • keywords backwards-compatibility removed
  • owner set to boomfish
  • status changed from new to assigned

Appending of generatedViewMapping to viewMappings should not be done in ModelGlueConfiguration.cfc as modules and actionpacks may load additional view mappings.

Appending of generatedViewMapping to viewMappings is best done in ViewRenderer.cfc so that it will always be the last mapping used.

Changed 16 years ago by boomfish

  • status changed from assigned to closed
  • resolution set to fixed

Fixed in SVN commit 248.

I took a different approach to the ones discussed here so far. The appending need only be done once, but only after the primary module and all its sub-modules have been loaded. I therefore resurrected the scaffolding module and added a listener for the modelglue.readyForModuleLoading message.

Note: See TracTickets for help on using tickets.