Alex Barth wrote a nice additional module to visualize all the data that lives in feedapi_stat
I do not attach it as a patch because CVS is awful when you need to add a new directory a patch. So i just attach a tarball w/ the new files.
There are two questions before committing:
1) Is it possible to theme_sparkline() ? This would solve a bug related to the files/sparkline directory. This is not created at install, but the custom theme function in feedapi_stat does not take care of that.
2) FeedAPI Stat or FeedAPI Stats? The name of the module "feedapi_stat".

Should be done before feedapi 1.9 beta

CommentFileSizeAuthor
feedapi_stat.tar_.gz2.18 KBaron novak

Comments

alex_b’s picture

alex_b commented

Cool- can you do a patch?

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented

#1: any benefit of having a patch instead of a tarball when it's about a single new directory? "CVS is awful when you need to add a new directory a patch" Do you know any better way of including brand new directories, than "cvsdo add" the new files in the main directory and the manually edit the patchfile carefully ?

alex_b’s picture

alex_b commented

#2: umpf, yeah. I forget.

The benefit would have been that I could have cleanly applied this patch to the copy of feedapi in my current project and keep track of it as such :p

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
Status: Active » Fixed

Named FeedAPI Statistics
Also theme('sparkline'... ) is used instead of custom code.

Status: Fixed » Closed (fixed)

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