dev/joomla#36 - Fix fatal error in joomla with civicrm task query param

[jitendrapurohit]

Overview

Fix fatal error in joomla with civicrm task query param

Before

CiviCRM Urls with task in it fails to load the page. To replicate

• Search for contributions - https://cividemo.com/administrator/?option=com_civicrm&task=civicrm/contribute/search&reset=1
• Try to send a receipt using the more link.

• The URL that is formed is

https://cividemo.com/administrator/?option=com_civicrm&task=civicrm/contribute/task&reset=1&task=receipt&id=12

Note there are 2 task query params

task=civicrm/contribute/task

and

task=receipt

After

Replace task with task_item

Comments

Gitlab - https://lab.civicrm.org/dev/joomla/-/issues/36

[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).

[seamuslee001]

@jitendrapurohit Why doesn't this falter on any of the other tasks in other contexts? Have you checked if the task forms will support the new key?

cc @eileenmcnaughton @lcdservices @mattwire

[jitendrapurohit]

There is no issue when the task param is in the session/request (which i think most of the action from the search forms work).

The problem with Send Receipt, Update Billing, Send Letter, etc is that these actions sets the task param in the URL (from the contact summary page), so the page is now loading with 2 task params in the URL eg

 /administrator/?option=com_civicrm&task=civicrm/contribute/task&reset=1&task=receipt&id=12

• task=civicrm/contribute/task is set by Joomla
• task=receipt is set by CiviCRM.

I think in this case, task=receipt overrides the original task civicrm/contribute/task and eventually fails to load the page.

---

Just to clarify -

Search form task dropdown already works fine.

It is only an issue when these task pages are loaded from contact summary page (i.e, for a single contact).

[eileenmcnaughton]

OK - I get it - yeah - we need to change 'task' to something else - this will also affect search kit

I think this is OK & it covers the lines of code I found - @colemanw do you see any problems with search kit

[colemanw]

Looks good to me. I believe SearchKit will adjust to the change without needing any updates.

[eileenmcnaughton]

OK - let's merge it