Search and Reporting - archived issueshttps://lab.civicrm.org/dev/report/-/issues2022-04-22T15:52:52Zhttps://lab.civicrm.org/dev/report/-/issues/1Contribution Detail: selecting the "soft credits" colum results in fatal error2022-04-22T15:52:52ZbgmContribution Detail: selecting the "soft credits" colum results in fatal errorTo reproduce:
* Go to the Reports > Contribution Reports > Contribution Details
* In the "columns", select the "soft credit" column
* Click "preview"
Results in a fatal error.
![Capture_d_écran_de_2018-03-07_16-58-43](/uploads/3a04f30...To reproduce:
* Go to the Reports > Contribution Reports > Contribution Details
* In the "columns", select the "soft credit" column
* Click "preview"
Results in a fatal error.
![Capture_d_écran_de_2018-03-07_16-58-43](/uploads/3a04f302cd865b4a013a2322a5f1c4c8/Capture_d_écran_de_2018-03-07_16-58-43.png)https://lab.civicrm.org/dev/report/-/issues/15Regression: Can't use operators to filter contact subtypes other than "Is One...2022-04-22T17:01:30ZJonGoldRegression: Can't use operators to filter contact subtypes other than "Is One Of"This is introduced by core#544, which introduces a hard-coded `LIKE` where the code should allow for different operators.
[A commit based on 5.14 is here](https://github.com/MegaphoneJon/civicrm-core/commit/625b0b06e64d02883614e31bb4ef7...This is introduced by core#544, which introduces a hard-coded `LIKE` where the code should allow for different operators.
[A commit based on 5.14 is here](https://github.com/MegaphoneJon/civicrm-core/commit/625b0b06e64d02883614e31bb4ef79825a1b8ad2#diff-d355cdb00cea3915a3cf306c6c08f6a6L2131) and hopefully I (or someone) will have time to merge and add tests in the near future.5.16.0https://lab.civicrm.org/dev/report/-/issues/16Unreleased regression - fee levels incorrectly show sold out (in code that wi...2022-04-22T15:53:35ZeileenUnreleased regression - fee levels incorrectly show sold out (in code that will be 5.16)Fees are incorrectly showing as sold out, blocking change fee selection.
This is an unexpected consequence of
https://github.com/civicrm/civicrm-core/pull/14244
The path is that because $this->_id is now set it gets assigned to the fo...Fees are incorrectly showing as sold out, blocking change fee selection.
This is an unexpected consequence of
https://github.com/civicrm/civicrm-core/pull/14244
The path is that because $this->_id is now set it gets assigned to the form
https://github.com/civicrm/civicrm-core/blob/5774b54f47445de233daa85672ce4f793dff347a/CRM/Event/Form/Participant.php#L265
Which then gets passed to the call to load the fee block
https://github.com/civicrm/civicrm-core/blob/c4145dedecb1f3157ecf8fd85421f562e8128e73/templates/CRM/Event/Form/Participant.tpl#L411
which results in $_pid being set & as a result online being set
https://github.com/civicrm/civicrm-core/blob/90b461f1623e75e94e0f472a3c6bf23e01defbc1/CRM/Event/Form/EventFees.php#L350
which leads to the element being frozen
https://github.com/civicrm/civicrm-core/blob/90b461f1623e75e94e0f472a3c6bf23e01defbc1/CRM/Event/Form/EventFees.php#L393
which is interpretted as 'sold out'
https://github.com/civicrm/civicrm-core/blob/6b83d5bdd0f2ca546924feae6aa42aeddb1d40cf/templates/CRM/Price/Form/PriceSet.tpl#L93
This is an example of a code antipattern which is too prevalent in our codebase - ie. hanging various assumptions off a parameter.
I *think* the right answer is to assign a variable of online & then pass that through & make appropriate decisions based on that.
The first question is - do we revert https://github.com/civicrm/civicrm-core/pull/14244 out of the rc and then re-commit into master to give us more time given how awful this code path is5.16.0https://lab.civicrm.org/dev/report/-/issues/67"You do not have permission to access this page" when exporting participants ...2022-04-22T15:52:34ZBobS"You do not have permission to access this page" when exporting participants in Search Kit- From Search Kit, create a new search.
- Search For: Participants
- Click Search and select 1 or more participants.
- From Action box: Choose Export Participants.
- Receive popup: "Error: You do not have permission to access this page."...- From Search Kit, create a new search.
- Search For: Participants
- Click Search and select 1 or more participants.
- From Action box: Choose Export Participants.
- Receive popup: "Error: You do not have permission to access this page."
I can, however, export participants from the traditional Search | Participants.
Confirmed on Demo site, Civi 5.38.0.https://lab.civicrm.org/dev/report/-/issues/93SearchKit: URLs to external sites no longer render2022-04-22T15:53:50ZJonGoldSearchKit: URLs to external sites no longer renderSearchKit displays that used to link to an external site now link to an invalid internal URL. This is a 5.44 regression.
### (Simplest) Steps to Replicate
* Create a new SearchKit search on Website entities. (Screenshot 1347)
* Create ...SearchKit displays that used to link to an external site now link to an invalid internal URL. This is a 5.44 regression.
### (Simplest) Steps to Replicate
* Create a new SearchKit search on Website entities. (Screenshot 1347)
* Create a new display, linking the website to itself. (Screenshot 1348).
* Click Preview.
### Expected Result
Link of "https://www.example.org" links to "https://www.example.org".
### Actual Result
Link of "https://www.example.org" links to "http://dmaster.localhost/https://www.example.org".
`git bisect` points to [PR #21820](https://github.com/civicrm/civicrm-core/pull/21820), specifically commit ID `8fd58f5640`.JonGoldJonGold