Feeling Lucky
Language Change
PBF
?
[#36926] - [5.3] Make it Optional to Save Response from the GET Request Task Routine
- Pending
- Medium
- Build: 4.1-dev
- # 36926
- Diff
- ditsuke:make-get-response-saving-optional
User tests: Successful: Unsuccessful:
Addresses a concern from #36201.
Summary of Changes
- Saving responses from the GET Request is now optional and disabled by default.
- The saved result now uses the
tmp_dirfrom the global configuration.
Testing Instructions
- Make a task
G1with the GET Request routine, keep "Save Response" disabled. - Make another task
G2with "Save Response" enabled. - Trigger the tasks either by waiting or by manually running from the list view.
Actual result BEFORE applying this Pull Request
- There is no option to not save the response.
- Response is saved in the
tmp/directory blind to thetmp_dirconfiguration.
Expected result AFTER applying this Pull Request
- There is an option to not save the response making retention opt-in. Response for the
G1task is not saved while response for theG2task is saved. tmp_dirconfiguration is respected.
Documentation Changes Required
nil
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration Language & Strings Front End Plugins |
| Labels |
Added:
Language Change
?
|
||
| Title |
|
||||||
| Labels |
Added:
PBF
|
||
HLeithner
- comment
- 2 May 2023
This pull request has been automatically rebased to 5.0-dev.
MacJoom
- comment
- 26 Aug 2023
Needs to updated, Call to undefined method Joomla\Component\Scheduler\Administrator\Table\TaskTable::getCurrentUser()
HLeithner
- comment
- 26 Aug 2023
Needs to updated, Call to undefined method Joomla\Component\Scheduler\Administrator\Table\TaskTable::getCurrentUser()
is fixed in 5.0 already, but this pr conflicts and has to be resolved
HLeithner
- comment
- 30 Sep 2023
This pull request has been automatically rebased to 5.1-dev.
HLeithner
- comment
- 24 Apr 2024
This pull request has been automatically rebased to 5.2-dev.
| Title |
|
||||||
HLeithner
- comment
- 2 Sep 2024
This pull request has been automatically rebased to 5.3-dev.
| Title |
|
||||||
This pull request has automatically rebased to 4.2-dev.