Redux - удаление элемента из состояния

Question

У меня проблемы с удалением элемента из состояния.

У меня есть контейнер счетов , который отображает список счетов, выставленных в InvoiceItem .

Я использую mapStateToProps и действие / редуктор для выключения и извлечения всех счетов.

Затем я использую mapDispatchToProps в InvoiceItem, чтобы отправить действие на кнопку удаления.(Только для удаления из состояния, а не из базы данных).

Является ли это правильным использованием при работе с действиями Redux (Connect)?Я знаю, что из контекста React16 Context API я подхожу к вещам.

При отладке mapDispatchToProps с использованием console.log в файлах действий и редукторов полезная нагрузка отображается как UNDEFINED

Ниже мой код:

types.js

export const GET_INVOICES = "GET_INVOICES";
export const GET_INVOICE = "GET_INVOICE";
export const DELETE_INVOICE = "DELETE_INVOICE";
export const INVOICES_LOADING = "INVOICES_LOADING";

invoiceActions.js

import axios from "axios";
import { GET_INVOICES, DELETE_INVOICE, INVOICES_LOADING } from "./types";

// getInvoices is a large axios call that goes here. It works fine.
// Leaving it out in this example.

// Delete Invoice
export const deleteInvoice = invoice => dispatch => {
 console.log(invoice) // UNDEFINED
dispatch({
  type: DELETE_INVOICE,
  payload: invoice
});
};

// Invoices loading
export const setInvoicesLoading = () => {
 return {
  type: INVOICES_LOADING
 };
};

invoicesReducer.js

import {
 GET_INVOICES,
 DELETE_INVOICE,
 INVOICES_LOADING
 } from "../actions/types";

const initialState = {
invoices: [],
loading: false
};

 export default function(state = initialState, action) {
  switch (action.type) {
    case INVOICES_LOADING:
     return {
      ...state,
      loading: true
     };
   case GET_INVOICES:
     return {
     ...state,
     invoices: action.payload,
     loading: false
    };
   case DELETE_INVOICE:
    return invoices.filter(invoice => invoice.Id !== invoice);
    default:
    return state;
   }
  }

InvoiceItem.js

import React, { Component } from "react";
import PropTypes from "prop-types";
import { connect } from "react-redux";
import { deleteInvoice } from "../../actions/invoiceActions";

class InvoiceItem extends Component {
 onDeleteClick(e) {
 this.props.deleteInvoice();
 }
 render() {
  const { invoice } = this.props;

    return (
     <div className="card card-body bg-light mb-3">
       <div className="row">
         <div className="col-lg-6 col-md-4 col-8">
           <h3>Invoice: {invoice.InvoiceNumber}</h3>
           <p>Client: {invoice.Contact.Name}</p>
           <button
            onClick={this.onDeleteClick.bind(this)}
            className="btn btn-danger">
            Remove
           </button>
      </div>
    </div>
  </div>
  );
  }
 }

 InvoiceItem.propTypes = {
 invoice: PropTypes.object.isRequired,
 deleteInvoice: PropTypes.func
 };

 const mapDispatchToProps = dispatch => {
   return {
     deleteInvoice: invoice => dispatch(deleteInvoice(invoice))
    };
  };

  export default connect(
   mapDispatchToProps,
   { deleteInvoice }
  )(InvoiceItem);

Invoices.js

import React, { Component } from "react";
import { connect } from "react-redux";
import PropTypes from "prop-types";
import Spinner from "../common/Spinner";
import InvoiceItem from "./InvoiceItem";
import { getInvoices } from "../../actions/invoiceActions";

class Invoices extends Component {
  componentDidMount() {
  this.props.getInvoices();
}

render() {
  const { invoices, loading } = this.props.invoice;
  let invoiceItems;

  if (invoices === null || loading) {
   invoiceItems = <Spinner />;
  } else {
  if (invoices.length > 0) {
    invoiceItems = invoices.map(invoice => (
      <InvoiceItem key={invoice.Id} invoice={invoice} />
    ));
  } else {
    invoiceItems = <h4>No invoices found...</h4>;
  }
}

return (
  <div className="profiles">
    <div className="container">
      <div className="row">
        <div className="col-md-12">
          <h1 className="display-4 text-center">Invoices</h1>
          {invoiceItems}
        </div>
      </div>
    </div>
  </div>
  );
 }
}

Invoices.propTypes = {
 getInvoices: PropTypes.func.isRequired,
 invoice: PropTypes.object.isRequired
};

const mapStateToProps = state => ({
 invoice: state.invoices
});


export default connect(
 mapStateToProps,
 { getInvoices }
)(Invoices);

Answer 1

deleteInvoice ожидает, что invoice объект будет передан. Поскольку вы ничего не передаете в данный момент, он регистрирует undefnied

onDeleteClick(e) {
   this.props.deleteInvoice(this.props.invoice);
}