Started using the 2.4 branch and noticed that the menu callbacks showing all node types even though I had only selected to show blog node types in the archive pages. Here is a patch that will fix the issue. The patch was created using git but I'm sure you can apply it just the same as if it were from CVS. The main problem was that you were only using array_keys and not filtering the array prior to doing so. The array that gets returned looks like this:

array(
'type1' => 'type1',
'type2' => 0
'type3' => 0
)

By filtering the array first you get rid of all the node types which were not selected and then you grab the keys only of the ones that are.

Hope this helps :)

CommentFileSizeAuthor
#1 week.patch1.7 KBBlakeLucchesi
week.patch1.75 KBBlakeLucchesi

Comments

BlakeLucchesi’s picture

BlakeLucchesi commented
StatusFileSize
new week.patch1.7 KB

Sorry, I had left a debug function call in there to output the array. This should be a better patch to apply.

Prometheus6’s picture

Prometheus6 commented
Assigned: Unassigned » Prometheus6

Damn...I caught that in the Month Archive module.

Prometheus6’s picture

Prometheus6 commented

Not ignoring you, got paid work to do.

Prometheus6’s picture

Prometheus6 commented
Status: Needs review » Fixed

Thanks for waiting. I did have the same problem with the montharchive module and had done something equivalent to what you did in your patch, so I wound up doing the same thing here.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.