fix: match Order

[Mehrbod2002]

Closes: #394

[plafer]

Thank you for submitting your first contribution!

Although @DaviRain-Su suggested doing this in the issue, this is not really what I had in mind. The goal is to remove the order_matches method.

I looked into it again, and I believe we should change all calls of the form

if chan_end.order_matches(&Order::Ordered) {
  /* if branch contents */
} else {
  /* else branch contents */
}

to

match chan_end_on_b.ordering {
    Order::Ordered => {
        /* if branch contents */
    },
    Order::Unordered => {
        /* else branch contents */
    }
    Order::None => {
        /* do nothing */
    }
}

, and then remove ChannelEnd::order_matches.

This matches the ibc-go behavior.

[plafer]

Closing as stale; feel free to open a new PR with the abovementionned suggestions applied!