This kind of flies in the face of the trend to separate stores/state from the components in favor of simpler components... Also, there's no cleanup of the event registrations which will lead to memory leaks.
Perhaps you're right - but the trend doesn't necessarily account for all use cases.
Assuming the OP is only modifying the components internal state (which has no bearing on the application state), I'd be all for it.