Prompt Anymous Users to Log In to View Video [#69653]

It was suggsested that I post this here. The reasoning was that if only authenticated users can access a video, anyonymous users should be advised of this automatically.

I just installed the Video module and it appears to work fairly well. The only problem I have is that only authenticated users will have access to the videos on my site. I set the permissions correctly in access_control and only authenticated users can actually view the videos.

However, when an anonymous user clicks on the video category I set up, and sees the list of available videos, there is no prompt telling him that he must login or register to see the videos. I know I can put that at the top of the page that lists the video categories, but I'd like for this message to appear on the individual video links that appear (as the login request appears for comments).

Is there a way to make a prompt appear on each link to let an anonymous user know he must login to access the videos?

Thanks.

-ron

Comment	File	Size	Author
#5	video_3.patch	2.17 KB	agilpwc
#5
#1	video_2.patch	1.68 KB	agilpwc
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

agilpwc CreditAttribution: agilpwc commented 23 June 2006 at 02:59

Assigned:	Unassigned	» agilpwc
Status:	Active	» Needs review

File	Size
video_2.patch	1.68 KB

Here is one possible solution, patch included on latest cvs from June 22. It creates another entry in access control 'play video'. So you give anoymous users 'access video' permission, but not 'play video'.

So they can read the description of the video, but not actually play it. They normal 'play' link is replaced by 'login or register to play video'

I use it on my own site http://agilityvision.com

Eric Larson

Comment #2

SomebodySysop CreditAttribution: SomebodySysop commented 24 June 2006 at 07:43

I'll get the latest cvs. Thanks!

Comment #3

fax8 CreditAttribution: fax8 commented 24 June 2006 at 17:44

@agilpwc

your patch looks good to me...
but you don't use drupal t() and l() functions for making hyperlinks.

Moreover indenting is incorrect.

Please correct this problem and I will commit this patch to cvs.

Fabio

Comment #4

SomebodySysop CreditAttribution: SomebodySysop commented 28 June 2006 at 00:05

I downloaded the cvs as of June 24th, and simply replaced the existing video directory with the new one. I don't see the "play video" option under access_control. Is there something else I need to do? I did not de- then re-install because I have existing videos I don't want to lose.

Suggestions?

Thanks.

-ron

Comment #5

agilpwc CreditAttribution: agilpwc commented 28 June 2006 at 00:26

File	Size
video_3.patch	2.17 KB

Here is the patch with nice indentation and use of t & l functions.

To answer the above question, the patch has not been included yet in cvs , it needs approval of the people in charge of the module.

Comment #6

fax8 CreditAttribution: fax8 commented 28 June 2006 at 18:49

Status:

Needs review

» Fixed

Committed to cvs. Thanks.

Next time please remove leading or heading blanks from strings
arguments to t(). This make translation easier.

t(' or ');

became

' ' . t('or') . ' ';

Thanks again.

Fabio

Comment #7

SomebodySysop CreditAttribution: SomebodySysop commented 29 June 2006 at 00:02

I have not tried the patched code, but when I downloaded the CVS code and tried to play videos, I would get garbled sound and no or frozen video.

What I did was simply copy the new video directory over my existing video directory. Is this the proper procedure for upgrading a prior release of the video module? The Drupal version is the same: 4.7.

Please advise. Thanks!