Framework: Replace flowRight occurences with compose for HOCs

[gziolo]

Description

In #3493 new utility method compose was introduced to be used with Higher-Order Components. This PR refactors all occurrences of flowRight around HOCs to be replaced with compose.

How Has This Been Tested?

Manually. All tests should still pass, linter should be happy and editor should work without errors.

Types of changes

Refactoring only. No changes in the existing functionalities.

Checklist:

• My code is tested.
• My code follows the WordPress code style.
• My code follows has proper inline documentation.

[youknowriad]

What about flow usage, I think we have some as well :)

[gziolo]

So I have just learned that flow and flowRight were used as if they were the same. That makes replacing flow easier and validates the need of compose helper method :)

[aduth]

So I have just learned that flow and flowRight were used as if they were the same.

That's not entirely true; in many cases we're assuming props to be passed from a previous HoC in the array, which works with flowRight, but not flow. See also: #2500 (comment)

[gziolo]

@aduth if you will take a closer look, you will notice that connect almost always goes first and it doesn't matter whether it's used with flow or flowRight. This is where my comment comes from. If you are aware of any place where I should update the order, let me know.

[gziolo]

@aduth It might be an issue only here. I will flip the order to be safe.

[gziolo]

Order reversed 👍