[4.1] Unification of the language com_scheduler, part 3

[tecpromotion]

Summary of Changes

During the translation into German and testing, an error occurred in the display of the duration time.

This PR shows the correct output.

Testing Instructions

create a new taks an click Run Test.

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Documentation Changes Required

.

[brianteeman]

ouch - I made that change and none of the testers spotted it

[tecpromotion]

ouch - I made that change and none of the testers spotted it

yes, unfortunately i didn't test that enough with my team when beta1 was released. but we should test it thoroughly again before the stable release.

[Quy]

I have tested this item ✅ successfully on 8474a84

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36297.}

[Quy]

RTC

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36297.}

[khu5h1]

I have tested this item ✅ successfully on 8474a84

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36297.}

[Shubhamverma2796]

I have tested this item ✅ successfully on 8474a84

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36297.}

[wojsmol]

@tecpromotion Please use %f instead.

[richard67]

@tecpromotion Could you have a look on the review suggestions and check if "%f" works, and if so, change to that? I'd like to come forward here, but I'm a bit reluctant to set RTC as long as there are open review discussions.

[tecpromotion]

@tecpromotion Could you have a look on the review suggestions and check if "%f" works, and if so, change to that? I'd like to come forward here, but I'm a bit reluctant to set RTC as long as there are open review discussions.

I can check this later.

[tecpromotion]

@tecpromotion Please use %f instead.

@wojsmol Sorry but '%f' does not work here.

joomla-cms/build/media_source/com_scheduler/js/admin-view-run-test-task.es6.js

Line 60 in b76ec55

modal.querySelector('.modal-body > div').innerHTML += `<div>${Joomla.Text._('COM_SCHEDULER_TEST_RUN_DURATION').replace('%s', output.data.duration.toFixed(2))}</div>`;

Here a '%s' is searched for and replaced.

Change both to %f does not work.

[richard67]

RTC

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36297.}

[bembelimen]

Thx

[alikon]

merged ? !!!!
why ? 🤕
evidently feedback from some are not welcomed or better simply ignored

[richard67]

@alikon Why ignored? @tecpromotion has checked with "%s" as requested and has reported back why we can't do that with the js as it is now.

[alikon]

this doesn't prove it's correct

[tecpromotion]

this doesn't prove it's correct

You're absolutely right, but that doesn't solve the problem.
A developer has to sit down and fix it everywhere.
Now there is an output and no more error.