Hi,
I am currently using module Address Field 7.x.1.1 and payumoney 7.x.1.0

I am getting several warnings

Warning: Missing argument 1 for addressfield_default_values(), called in /var/www/html/sandbox/sites/all/modules/contrib/payumoney/modules/pmphone/commerce_payumoney_pmphone.module on line 24 and defined in addressfield_default_values() (line 374 of /var/www/html/sandbox/sites/all/modules/addressfield/addressfield.module).

Warning: Missing argument 2 for addressfield_default_values(), called in /var/www/html/sandbox/sites/all/modules/contrib/payumoney/modules/pmphone/commerce_payumoney_pmphone.module on line 24 and defined in addressfield_default_values() (line 374 of /var/www/html/sandbox/sites/all/modules/addressfield/addressfield.module).

Warning: Creating default object from empty value in commerce_payumoney_transaction() (line 212 of /var/www/html/sandbox/sites/all/modules/contrib/payumoney/commerce_payumoney.module).

To fix above warning I have create patch.

CommentFileSizeAuthor
#2 payumoney-several_warnings-2536552-1.patch2.91 KBshashikant_chauhan
payumoney-several_warnings-1.patch3.19 KBshashikant_chauhan

Comments

Status: Needs review » Needs work

The last submitted patch, payumoney-several_warnings-1.patch, failed testing.

shashikant_chauhan’s picture

shashikant_chauhan commented
Status: Needs work » Needs review
StatusFileSize
new payumoney-several_warnings-2536552-1.patch2.91 KB

Adding patch again

sarathkm’s picture

sarathkm
Sapiens
Primary language English
Location Bengaluru, Karnataka
commented
Status: Needs review » Reviewed & tested by the community

Thanks :)

sarathkm’s picture

sarathkm
Sapiens
Primary language English
Location Bengaluru, Karnataka
commented
Status: Reviewed & tested by the community » Needs review
sarathkm’s picture

sarathkm
Sapiens
Primary language English
Location Bengaluru, Karnataka
commented

Seems like this patch not working for me. The errors are gone but shows some sort of error while redirecting.

sarathkm’s picture

sarathkm
Sapiens
Primary language English
Location Bengaluru, Karnataka
commented

:( Test URL seems to be not working but production does :)

sarathkm’s picture

sarathkm
Sapiens
Primary language English
Location Bengaluru, Karnataka
commented
Status: Needs review » Reviewed & tested by the community

Thanks :)

  • 4810b4a committed on 7.x-1.x 
    Issue #2536552 by shashikant_chauhan: Warning: Missing argument 1 for...
sarathkm’s picture

sarathkm
Sapiens
Primary language English
Location Bengaluru, Karnataka
commented

Pushed code to repository.

shashikant_chauhan’s picture

shashikant_chauhan commented

Hi Sarath,
Can you add credits to me for this issue?
Commit is not showing in my profile.
More info : https://www.drupal.org/node/52287

sarathkm’s picture

sarathkm
Sapiens
Primary language English
Location Bengaluru, Karnataka
commented

Hi Shashikant,

Yes I did as specified in give credit section.

View Commits

sarathkm’s picture

sarathkm
Sapiens
Primary language English
Location Bengaluru, Karnataka
commented
Status: Reviewed & tested by the community » Fixed

  • sarathkm committed 3b87756 on 7.x-1.x 
    Issue #2536552 by shashikant_chauhan: Warning: Missing argument 1 for...
sarathkm’s picture

sarathkm
Sapiens
Primary language English
Location Bengaluru, Karnataka
commented

Hi Shashikant,
This time Done. Thanks Again for the patch :)

Status: Fixed » Closed (fixed)

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

shashikant_chauhan’s picture

shashikant_chauhan commented