Swapping through low-liquidity pairs

[aterehov]

Bug Description Website offers to swap some tokens through low-liquidity pairs while there are high-liquidity pairs for the same tokens. Examples: For BNB to CAKE website offers to use BNB > BUSD > CAKE, while the better route is BNB > CAKE (directly), because there is an $11M pair for that. USDT to NYA swap: website offers USDT > WBNB > NYA, while the better route is USDT > WBNB > CAKE > NYA (more liquidity there)

Steps to Reproduce

1. Go to exchange.pancakeswap.finance
2. Set a swapping route: from USDT to NYA (it shows out the bug more clearly and appears more often)
3. Enter "20" in the top field (USDT)
4. You will see the wrong route USDT > WBNB > NYA ...

Expected Behavior A correct route for USDT > NYA example is USDT > WBNB > CAKE > NYA (or any other through CAKE). The same is for other routes: it should rather choose high-liquidity pairs.

Additional Context Some screenshots showing examples above with wrong routes

[geckah]

Having the same issue with eth - compound. It goes through bnb resulting in high price impact

[RabbitDoge]

The eth/comp bug is fixed. I'll check for the cake pair.

[masha256]

I'm seeing the same for BUSD to ETH. It wants to route BUSD -> WBNB -> ETH when there is a BUSD/ETH LP with $1.9M USD worth of liquidity in it. This results in paying twice the fee on a swap since it routes through two pools instead of one.