Since the channel shows a video rather than an obvious way to go straight to the Youtube channel, the additional (optional) display of a link would be nice.

CommentFileSizeAuthor
#5 youtubechannel-add_an_optional_link_to_the_channel_in_the_output-1983592-5.patch1.9 KBNitinSP
#3 Add_an_optional_link_to_the_channel_in_the_output-1983592-3.patch854 bytesNitinSP
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

adraskoy’s picture

adraskoy CreditAttribution: adraskoy commented

I solved this one by adding a channel variable to hand to the template in youtubechannel_getview():
$vars['channel'] = check_url("http://www.youtube.com/user/{$youtube_user}");

This required fixing bug 1983774 to make the template overridable.

parwan005’s picture

parwan005 CreditAttribution: parwan005 commented
Issue summary: View changes

Thanks adraskoy for your suggestion. Will try to implement it in future release.

NitinSP’s picture

NitinSP CreditAttribution: NitinSP at Clarion Technologies commented
FileSize
Add_an_optional_link_to_the_channel_in_the_output-1983592-3.patch854 bytes

Add youtube channel link in youtube channel block. So if user click on it will redirect to respective youtube user page.

NitinSP’s picture

NitinSP CreditAttribution: NitinSP at Clarion Technologies commented
Status: Active » Needs review
NitinSP’s picture

NitinSP CreditAttribution: NitinSP at Clarion Technologies commented
FileSize
youtubechannel-add_an_optional_link_to_the_channel_in_the_output-1983592-5.patch1.9 KB

Updated formatting, please review the latest patch #5.

hiramanpatil’s picture

hiramanpatil CreditAttribution: hiramanpatil at Clarion Technologies commented
Status: Needs review » Reviewed & tested by the community

Thanks for the patch. I have tested it and its working fine.

NitinSP’s picture

NitinSP CreditAttribution: NitinSP at Clarion Technologies commented

commit and merge this changes in 7.x-2.0.

NitinSP’s picture

NitinSP CreditAttribution: NitinSP at Clarion Technologies commented
Status: Reviewed & tested by the community » Fixed

  • NitinSP authored 00d34e8 on 8.x-1.x 
    Issue #1983592 by NitinSP: Add an optional link to the channel in the...

Status: Fixed » Closed (fixed)

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