Project60 / Project60/org.project60.banking
Relative links not working cross-platform
Nobody has claimed this yet.
- Dominant language
- PHP
- Stars
- 22
- Forks
- 45
- Avg merge
- 3d 15h
- Merged PRs (30d)
- 7
Description
Viewing the transactions list (example https://my.crm/wp-admin/admin.php?page=CiviCRM&q=civicrm%2Fbanking%2Fpayments&status_ids=8222%2C8223&s_list=1&show=payments ) shows two links: party (which links to contact if found) and view (transaction).
The link for party reads:
https://my.crm/civicrm/contact/view?reset=1&cid=63
This links to frontend view of the contact.
Correct link for WordPress should be:
https://my.crm/wp-admin/admin.php?page=CiviCRM&q=civicrm%2Fcontact%2Fview&reset=1&cid=63
I guess this is no problem in Drupal, but annoying for WordPress users.
Seems to be line 126 in https://github.com/Project60/org.project60.banking/blob/master/templates/CRM/Banking/Page/Payments.tpl
I'm sorry that I can not help creating a variable for this link (working for Drupal / StandAlone AND WordPress).
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at line 126 of templates/CRM/Banking/Page/Payments.tpl, where the party link is built for the transactions list. Compare the shown frontend URL with the required WordPress admin URL and check how links must differ across Drupal, StandAlone, and WordPress. Done means the party link opens the contact correctly on each supported platform.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- web-dev
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100