I am reviewing the latest from 5.x-3.x-dev branch and not having much luck displaying activities, page or block.

The problem it seems is that default theming functions do not really theme 3.x activities (I generated a few I see in DB). I figure everyone who uses the module has overridden them and therefore does not experience the problem.

We need some good defaults committed to CVS, for both page and block. Could anyone contribute their code?

CommentFileSizeAuthor
#1 activity-screenshot.jpg130.71 KBjaydub

Comments

jaydub’s picture

jaydub commented
StatusFileSize
new activity-screenshot.jpg130.71 KB

hmmm, I am able to see page and block views with my activity...

dkruglyak’s picture

dkruglyak commented
Title: Default theming completely broken » Need README file to explain installation process
Component: User interface » Documentation

Never mind, I tracked down the problem, which is documented in activity_admin_settings comment and on the module front page.

I think perhaps the module needs README to remind about the correct installation steps, such as saving settings.

sirkitree’s picture

sirkitree commented

There is a install readme on the project page. I'll throw this into a readme file as soon as I get a chance.

There is a note on there to make sure you save the tokens on /admin/settings/activity for activity to show up.

I'm working on fixing this by inserting the defaults with a variable_set on the installation process of the contribs.

dkruglyak’s picture

dkruglyak commented
Title: Need README file to explain installation process » Improve installation process: README and/or better default settings
Component: Documentation » Miscellaneous
Category: bug » task

Great. I also found that many of default tokens do not make much sense or are in need of translation.

E.g. in nodeactivity module, operations get recorded by nodeapi hook name ("insert", "update", "delete", ...), while configuration UI says different. This means when node is Created (as Settings UI says) the default token says it was "inserted". Another problem is translation of node-type into user readable label (e.g. "feed_item" instead of "Feed Item").

We should come up with better tokens & labels for default installs.

sirkitree’s picture

sirkitree commented

Agreed. There is a patch submitted that I haven't tested out myself just yet for the human readable format here: http://drupal.org/node/223021

jaydub’s picture

jaydub commented
Version: 5.x-3.x-dev » 5.x-4.x-dev
Component: Miscellaneous » Documentation
Assigned: Unassigned » jaydub
Priority: Critical » Normal
sirkitree’s picture

sirkitree commented
Status: Active » Closed (won't fix)

5.x unsupported