Remove `ChannelEnd::order_matches()`

[plafer]

We should use a match statement instead. This pattern encourages not looking at all possible variants, and shouldn't be used.

At the same time, we should understand what the Order::None represents, and if we keep it, what to do in cases where the order is None.

[DaviRain-Su]

I think is ok, maybe can implement to seem this.

  pub fn order_matches(&self, other: &Order) -> bool {
        // self.ordering.eq(other)
        match (self.ordering, other) {
            (Order::None, Order::None) => true,
            (Order::Unordered, Order::Unordered) => true,
            (Order::Ordered, Order::Ordered) => true,
            _ => false,
        }
   }

[DaviRain-Su]

Just search order_matches(), for now, pass param values is Order. We don't used any field.