dev/core#346 - Online Pay Now error when used from dashboard checksum…

[jitendrapurohit]

… link

Overview

Fix checksum URL for paynow link when dashboard is accessed through checksum.

Before

PayNow button URL is not built with cs param when user dashboard is access with checksum. More details at https://lab.civicrm.org/dev/core/issues/346

After

PayNow button URL has cs param which enabled user to complete the payment.

Comments

@magnolia61 Can you pls test this change?

---

Gitlab Issue - https://lab.civicrm.org/dev/core/issues/346

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[mlutfy]

jenkins, test this please

[jitendrapurohit]

Updated the PR to remove inaccessible links/section for checksum users. Display of dashboard looks like -

[magnolia61]

Thanks Jitendra. Tested your updated PR and it works fine and as expected.
BTW I used your screenshot to clarify my final remark in this #12319 PR and this take partial payments, user dashboard and checksum access and Pay Now functionality a big step further! Thanks!

[eileenmcnaughton]

Merging based on @magnolia61 testing

[magnolia61]

Since this solves a bug resulting in a fatal error for end users in CivCRM 5.4.0 Would it be possible to merge this in the upcoming 5.5?