Skip to content

Commit

Permalink
Remove unused method arg in RelayReader (#4327)
Browse files Browse the repository at this point in the history
Summary:
This arg is not used any more

Pull Request resolved: #4327

Reviewed By: voideanvalue

Differential Revision:
D46361028

Privacy Context Container: L1125407

Pulled By: captbaritone

fbshipit-source-id: 538c93609ab08cd3bc1dafefc5487845f6fd32a7
  • Loading branch information
captbaritone authored and facebook-github-bot committed Jun 1, 2023
1 parent 0e33bf2 commit e9150cc
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions packages/relay-runtime/store/RelayReader.js
Original file line number Diff line number Diff line change
Expand Up @@ -281,11 +281,7 @@ class RelayReader {
return this._variables[name];
}

_maybeReportUnexpectedNull(
fieldPath: string,
action: 'LOG' | 'THROW',
_record: Record,
) {
_maybeReportUnexpectedNull(fieldPath: string, action: 'LOG' | 'THROW') {
if (this._missingRequiredFields?.action === 'THROW') {
// Chained @required directives may cause a parent `@required(action:
// THROW)` field to become null, so the first missing field we
Expand Down Expand Up @@ -332,7 +328,7 @@ class RelayReader {
if (fieldValue == null) {
const {action} = selection;
if (action !== 'NONE') {
this._maybeReportUnexpectedNull(selection.path, action, record);
this._maybeReportUnexpectedNull(selection.path, action);
}
// We are going to throw, or our parent is going to get nulled out.
// Either way, sibling values are going to be ignored, so we can
Expand Down

0 comments on commit e9150cc

Please sign in to comment.