I'm making (yet again) a 'Most Emailed' block on my current project, and after including in this fine module discovered that views isn't supported yet. Are you planning to support views in the near future, or is it more likely if I can come up with a patch to add views support?

CommentFileSizeAuthor
#4 mw.patch1.11 KBmoshe weitzman

Comments

seanr’s picture

Assigned: Unassigned » seanr

It's there in the 5.x version, but I haven't gotten 6.x caught up yet. I suppose I should get working on that now. LOL

seanr’s picture

Status: Active » Postponed

ARGH! This is going to take a while. Merlin went and COMPLETELY changed the views hooks, and has't finished the update docs yet (since Views 2 is still in beta), so it's proving to be a bit overwhelming. All of the other Drupal 5 improvements are there now though, so I'd recommend pulling the latest code from CVS (or get the package after it updates tonight).

seanr’s picture

Status: Postponed » Active

I think I've got it all working now. Please grab the module from CVS (HEAD) and let me know how it works for you.

moshe weitzman’s picture

Status: Active » Needs review
StatusFileSize
new1.11 KB

Small patch updates for released views. just adds the API function which is required.

I also fixed what looks like a braino to me in the sorting of the Most Emailed block.

catch’s picture

Status: Needs review » Reviewed & tested by the community

Ran into both these issues and Moshe's patch is good.

seanr’s picture

Thanks, Moshe and Catch. Done and committed to HEAD and DRUPAL-6--1.

seanr’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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