'location' prop not changing in a component with 'withRouter'

Version

4.1.2

Test Case

https://codesandbox.io/s/61m9v82r7n

Steps to reproduce

Navigate between Home and Home + query.

Fix

The problem can be fixed by changing Child from being a PureComponent to Component. This is very weird as it has absolutely no connection to either being a child of the <Route /> component, or being the one that is decorated with withRouter.

Expected Behavior

I expect DeepChild’s location prop to change whenever the URL changes.

Actual Behavior

DeepChild’s prop is not changing despite it being correctly connected to withRouter.

Note

The setup, together with the connect usage, is exactly how I encountered it in my project. I did not test the issue without using connect.

[macrozone]

When a component says it doesn’t need to be updated, it is also speaking for its children, so if a child needs to update when the location changes, the parent shouldn’t be blocking it.

i think that’s not true. A component should never assume whether its children are components or connected containers:

<PureComponent>
   <Component1 />
   <Component2 />
</PureComponent>

Wether Component1 or Component2 should ever rerender is of no concern of PureComponent. Redux handles this case correctly: if Component1 is connected, it rerenders its children whenver the mapped state changes.

But if Component1 uses withRouter hoc, it does not re-render on location changes. I would expect withRouter to behave similar to connect in this regard.

[oreqizer]

I did read it, however I thought that wrapping a component in withRouter gets around any blocked updates

do I understand correctly that I need to wrap every single pure component in the hierarchy in withRouter or pass the location prop from the upmost parent so the changes propagate?