Needs review
Project:
Node Option Premium
Version:
7.x-1.1
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
7 Feb 2012 at 21:51 UTC
Updated:
20 Apr 2012 at 16:34 UTC
Jump to comment: Most recent file
Comments
Comment #1
davidwbarratt commentedHere's the patch! :)
Comment #2
anrikun commentedThanks!
Doesn't it need some hook_update_N() for migrating any existing default teaser settings to the newly added custom premium_teaser mode?
Comment #3
davidwbarratt commentedI added the update to copy the settings from the teaser to the premium_teaser. This should allow migration to the new type to happen smoothly and easily.
I also added a theme suggestion so the view mode can be themed independently. :)
I'm excited to see this committed!
Comment #4
anrikun commentedThanks again.
Just a detail: "premium_teaser" should be renamed to "nopremium_teaser" everywhere.
Comment #5
davidwbarratt commentedFixed! thanks!
Comment #6
davidwbarratt commentedAny update on when this might be commited?
thanks!
david barratt
Comment #7
davidwbarratt commentedFixed a bug in the patch that passed $view_mode = 'teaser' to the theme. Instead, the proper $view_mode = 'nopremium_teaser' has been set and theme can handle the view_mode accordingly.
Updated Patch is attached.
thanks!
david barratt