Hi,

Validation check: Undefined property & Warning: Invalid argument when you provide wrong api key/client ID etc..

CommentFileSizeAuthor
#5 youtubeimport-undefinedproperty-2745407-4.patch990 bytesiamfrisbee
#3 youtubeimport-undefinedproperty-2745407-2.patch10.91 KBharish b
youtube_importbug1.png121.75 KBharish b
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

harish b created an issue. See original summary.

harish b’s picture

harish b CreditAttribution: harish b at Srijan | A Material+ Company for Srijan | A Material+ Company commented
Issue summary: View changes
harish b’s picture

harish b CreditAttribution: harish b at Srijan | A Material+ Company for Srijan | A Material+ Company commented
Status: Needs work » Needs review
FileSize
youtubeimport-undefinedproperty-2745407-2.patch10.91 KB
harish b’s picture

harish b CreditAttribution: harish b at Srijan | A Material+ Company for Srijan | A Material+ Company commented
Issue tags: +code cleanup
iamfrisbee’s picture

iamfrisbee CreditAttribution: iamfrisbee commented
FileSize
youtubeimport-undefinedproperty-2745407-4.patch990 bytes

Created a new patch with fewer changes to the underlying code. Additionally, fewer spacing issues that the robot will not like. Good catch though. Thanks for testing. I'm also committing the change.

iamfrisbee’s picture

iamfrisbee CreditAttribution: iamfrisbee commented
Status: Needs review » Closed (fixed)