Skip to content

Fixed entry type casting to string when displaying an entry in the terminal#50

Open
juarezlive wants to merge 1 commit into
masterfrom
fix-6260-cli-value
Open

Fixed entry type casting to string when displaying an entry in the terminal#50
juarezlive wants to merge 1 commit into
masterfrom
fix-6260-cli-value

Conversation

@juarezlive

@juarezlive juarezlive commented Jun 10, 2026

Copy link
Copy Markdown

Description

This PR fixes an issue where displaying a field where the value is an array would be changed to the word "array" because the value is type casted to a string.

Related Issue

Closes: https://github.com/gravityforms/backlog/issues/6260

Testing

Testing Requirements

  • This needs to be tested in multiple browsers.
  • This needs to be tested with both Gravity and Orbital themes.
  • This needs to be tested with and without AJAX submission.
  • This needs accessibility testing.
  • This needs to be tested with different versions of PHP or database servers.
  • This needs to be tested with different versions of WordPress.
  • None of the above.

Testing Instructions

Using this branch of CLI.

  1. Create a form with a List field.
  2. Submit the form adding at least two items in the list.
  3. Open the site's Shell.
  4. Run the command: wp gf entry get [entry ID] --format=json
  5. Confirm that the entry displays the List field: {"ID":[fieldID],"Field":"List","Value":"[\"item 1\",\"item 2\",\"item 3\"]"}]

QA Tracking Spreadsheet

Screenshots

Checklist

  • I've checked the error log to ensure my code doesn't throw any errors.
  • I've added a changelog entry (if necessary) and updated the version number in 4 places, run npm install and committed package-lock.json (if necessary).
  • I have added a "Documentation Needed" label to this PR (if necessary).

@juarezlive juarezlive requested review from a team June 10, 2026 19:01
@gosiapado gosiapado requested review from gosiapado and removed request for a team June 12, 2026 13:54

@gosiapado gosiapado left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants