views needs to be enabled to show the "admin/reports/pop_links" link in the reports section.

I will create a patch for it to require views at install.

simple patch but just trying to help :)

Comment	File	Size	Author
#4	External_Link_Popularity-require-views-1343448-4.patch	337 bytes	scuba_fly
#2	requireViews-1343448-3.patch	337 bytes	scuba_fly
#1	requireViews-1343448-1.patch	310 bytes	scuba_fly

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Netherlands, Vinkeveen

CreditAttribution: scuba_fly commented 16 November 2011 at 14:03

File	Size
requireViews-1343448-1.patch	310 bytes

Here is the patch.

correct me if i'm wrong or making a patch for this is unnecessary.

Best,

Jack

Log in or register to post comments

Comment #2

Netherlands, Vinkeveen

CreditAttribution: scuba_fly commented 16 November 2011 at 14:28

File	Size
requireViews-1343448-3.patch	337 bytes

I noticed that views ui is also required ( otherwise errors wil show up )

here's the patch

Log in or register to post comments

Comment #3

Netherlands, Vinkeveen

CreditAttribution: scuba_fly commented 16 November 2011 at 14:29

Status:

Needs work

» Needs review

Log in or register to post comments

Comment #4

Netherlands, Vinkeveen

CreditAttribution: scuba_fly commented 21 January 2013 at 13:42

File	Size
External_Link_Popularity-require-views-1343448-4.patch	337 bytes

trying to re-submit patch #3 for auto testing.

Log in or register to post comments

Comment #5

Netherlands, Vinkeveen

CreditAttribution: scuba_fly at LimoenGroen commented 27 July 2017 at 10:18

Assigned:	scuba_fly	» Unassigned
Issue summary:	View changes

Log in or register to post comments