2

I have a reducer which returns an object with an array in the object. I render the list of items in the array and when the user clicks on that item I want to re-render the array (or at least the item). I've created a jsbin that shows the problem:

https://jsbin.com/fadudeyaru/1/edit?js,console,output

To reproduce the issue, click the + or - button a few times to create a history. then click on one of the history items. you'll notice that the console log notices the event, and updates the state, but the list is not re-rendered. This can be verified by clicking the +/- button again after clicking on an item in the list. after that you'll see that it renders correctly.

Question is why does react-redux not cause this to be re-rendered? is there something I need to do to force the issue?

Thanks in advance.

Ben
  • 16,124
  • 22
  • 77
  • 122
  • IIRC, returning an object from a reducer does only a shallow copy... perhaps the array isn't getting updated? – Kryten Sep 23 '16 at 19:02
  • no, it is getting updated... you can see this in both the console.log output as well as by selecting a few history items and then clicking the + or - button... – Ben Sep 23 '16 at 20:46

1 Answers1

6

State in redux is immutable. This means that reducer should create a new state for every mutation. Preferably, a deep clone should be done when arrays are present. The following code does an approximate deep clone for your code to work. Try utilities like lodash/deepClone for an easier solution.

const counter = (state = {count:0, history:[]}, action) => {
  let {count} = state;
  let history = [...state.history];

  switch (action.type) {
    case 'SELECT':
      history[action.i].selected = true;
      break;
    case 'INCREMENT':
      history.push({count,selected:false});
      count++;

      break;
    case 'DECREMENT':
      history.push({count,selected:false});
      count--;
      break;
    default:
      break;
  }
    console.log("count reducer: ", {count,history})
  return {count,history};
} 
Ben
  • 16,124
  • 22
  • 77
  • 122
vijayst
  • 20,359
  • 18
  • 69
  • 113
  • @Ben I have updated the code. This one works. Better to use lodash/deepClone to work on the cloned object. – vijayst Sep 24 '16 at 04:24