1- drupal_set_message and drupal_get_message is deprecated as per this link - https://www.drupal.org/node/2774931 so fixing those parts.

2- AffiliatesProduct name field length is 50 (which is very less as while fetching data from API, showing error)

3- Fixing template path

4- Fixing some minor issues in product entity to make it compatible with advance feeds crawler.

CommentFileSizeAuthor
#7 2980130-7.patch4.27 KBankitjain28may
#5 2980130-5.patch4.27 KBankitjain28may
#2 scraper-2980130-2.patch6.77 KBankitjain28may
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ankitjain28may created an issue. See original summary.

ankitjain28may’s picture

ankitjain28may CreditAttribution: ankitjain28may as a volunteer and at Google Summer of Code commented
Status: Active » Needs review
FileSize
scraper-2980130-2.patch6.77 KB
ankitjain28may’s picture

ankitjain28may CreditAttribution: ankitjain28may as a volunteer and at Google Summer of Code commented
Issue summary: View changes
ankitjain28may’s picture

ankitjain28may CreditAttribution: ankitjain28may as a volunteer and at Google Summer of Code commented
Title: Add fields in config form for Scraper and Fix some issues » Fix some issues in product entity
Status: Needs review » Active
ankitjain28may’s picture

ankitjain28may CreditAttribution: ankitjain28may as a volunteer and at Google Summer of Code commented
Status: Active » Needs review
FileSize
2980130-5.patch4.27 KB

Please review.

dbjpanda’s picture

dbjpanda CreditAttribution: dbjpanda commented
Status: Needs review » Reviewed & tested by the community

:)

ankitjain28may’s picture

ankitjain28may CreditAttribution: ankitjain28may as a volunteer and at Google Summer of Code commented
FileSize
2980130-7.patch4.27 KB

Just a change in the length of the name from 50 to 1024

thedrupalkid’s picture

thedrupalkid CreditAttribution: thedrupalkid as a volunteer commented

#7 Looks good we can go ahead and commit.

dbjpanda’s picture

dbjpanda CreditAttribution: dbjpanda commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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