alex-klock / razor-mediator-4-tridion

Automatically exported from code.google.com/p/razor-mediator-4-tridion
1 stars 4 forks source link

Patch for /trunk/Tridion.Extensions.Mediators.RazorMediator/Tridion.Extensions.Mediators.RazorMediator/Models/DynamicPackage.cs #5

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Modified TryGetMember so that it doesn't cache the values provided by the 
Tridion package. Modified AddPackageItem and renamed it to 
GetDynamicItemFromTridionPackageItem in support of the same goal. 

Caching these values is not helpful, because the value returned by the Tridion 
package can change within the lifetime of the cache. (You could consider 
surfacing .Remove() in the DynamicPackage, and using that to dirty the cache, 
although that would still leave possible problems with people accessing the 
package directly.) 

This patch works for me, but I'd suggest a further review rather than just 
following my approach. Ping me if I can help. 

Original issue reported on code.google.com by dominic....@gmail.com on 3 Dec 2012 at 3:56

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by alex@tahzoo.com on 4 Dec 2012 at 2:13

GoogleCodeExporter commented 9 years ago

Original comment by alex@tahzoo.com on 21 Dec 2012 at 2:22