Delete unnecessary code

[eopws]

Summary of Changes

Delete extra if block.
This pull request is not assigned with any issue as it makes the code more redable

Testing Instructions

No testing instruction is specified as the pull request doesn't change the application logic, but makes code more readable

[brianteeman]

What makes you think the code isnt required?

Where is it used? That way people will be able to test it.

[eopws]

What makes you think the code isnt required?

Because the code has inner if block with the same condition (!isset($paths[$dispatcherHash][$path])) as the outer if block has (!isset($paths[$dispatcherHash][$path]))

[brianteeman]

That's exactly the sort of information that should have been in your first post explaining the change. Remember that a diff/patch/ will not always show enough code for it to be obvious

[eopws]

Sory, if I provided not enough information. I thought this pull request has too small changes to write a lot to description.

[ReLater]

I have tested this item ✅ successfully on 44ba3bc

Code review. Obviously redundant code after checking line 247 (outer if{})

---

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

[joomdonation]

I have tested this item ✅ successfully on 44ba3bc

---

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

[Quy]

RTC

---

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

[eopws]

Does anybody know how to delete that second commit that I added accidentally?

[richard67]

@eopws I've reverted your last commit for you. Maybe one thing you should keep in mind when making again a pull request in future: Please always first create a new branch based on the branch for which the PR shall be made, which is in this case staging, and then use that new branch for the PR. Golden rule: For each PR a separate branch.

[HLeithner]

Thanks