Alternate module wrapper not working for News Module?

[jw0ollard]

I've done alternate wrappers a dozen times now, but I can't seem to do it for the News Module (and perhaps it may not work on any of the "default" modules?)

To be a little more specific, I have a few settings for the News Module customized on this page. Is it possible that when the module is customized on a page that it ignores ALL of the default settings.. meaning the alternative wrapper setting? (Since you can't customize that on each indiv. page)

I've even tried different wrapper code. I simply copied the default wrapper template, and changed ONE thing, to make sure it wasn't something wrong with my wrapper code.

-----

You know, what I think the issue is (and it just this second occurred to me) is that the code for grabbing the wrapper template is hardcoded into the news.php file. Doh! I remember seeing this a lot when making custom versions of the news.php file. I also don't think you can tell news.php to use different templates for the actual code either, but I'm not sure there's a way around that.

So, basically, it's either make the alternate module wrapper setting work with news.php, or remove the option?

[Brian]

Which version of the CMPS are you running? The wrapper template was hard coded into the file before, but that should have been corrected in v3.0 RC2.

[jw0ollard]

That's odd, as I'm running RC2.. And indeed, it does use "fetch_template($news_wrappername)" in the code.

I can't really test this anymore without messing up the work I've done on my News page.. I'd like to help figure out why it wasn't using my custom wrapper, but there's not really much else I can do.

Is there something basic I might be missing? I kept "Use Wrapper" set to "Yes", entered the name of my custom wrapper template in the input below it, and hit save. I've done this a dozen times before for my own custom modules and never had any problems.

But, anyway, I figured out a better way to do what I was planning, without even using the module wrapper, but if I ever need to use one again, I'll just hardcode the file.

----

I'm sure you know your own code, but maybe there's a reason that this line $news_wrappername = ($news['altshell']) ? $news['altshell'] : 'adv_portal_module_wrapper'; Is still evaluating false for "$news['altshell'] even if there is one?

[Brian]

Ahhh, looks like that would be the culprit. Not sure how it was working for me when I tested things, but this would be the correct code (just replacing $news with $newsmod)
$news_wrappername = ($newsmod['altshell']) ? $newsmod['altshell'] : 'adv_portal_module_wrapper';