There is no implementation of hook_views_default_views_alter() as done with other BW node types. This results in the documents not being counted in the All tab.

Patch follows.

CommentFileSizeAuthor
#1 2044827-commons-documents-bw-integration.patch749 bytesBarisW
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

BarisW’s picture

BarisW CreditAttribution: BarisW commented
Status: Active » Needs review
FileSize
2044827-commons-documents-bw-integration.patch749 bytes
RobKoberg’s picture

RobKoberg CreditAttribution: RobKoberg commented

On a fresh install it looks like the Document (and Notice) content type features are not enabled. So perhaps this is by design?

BarisW’s picture

BarisW CreditAttribution: BarisW commented

That's correct, but once you enable the module, it should pick up the integration with the Browsing Widget and thus add the number of nodes to the tabs.

If you have 3 blogs, 5 polls and 2 documents, the All tab will say 8 items. Which should be 10. This patch fixes that.

Devin Carlson’s picture

Devin Carlson CreditAttribution: Devin Carlson commented
Status: Needs review » Fixed

I thought that this might be by design but I agree that it's confusing that "All" displays and counts everything but documents. If documents shouldn't be counted in the All tab then they shouldn't have a tab in the browsing widget or the All tab should be retitled.

Committed #1 to Commons Documents 7.x-3.x.

http://drupalcode.org/project/commons_documents.git/commit/cb02804

Status: Fixed » Closed (fixed)

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