[PR #89] [CLOSED] Fix fatal error when dynamodb item is not found #113

New Issue

admin · 2026-01-23T11:35:15+01:00

admin commented

2026-01-23 11:35:15 +01:00

📋 Pull Request Information

Original PR: https://github.com/doctrine/KeyValueStore/pull/89
Author: @gustavobatalha
Created: 4/17/2018
Status: ❌ Closed

Base: master ← Head: master

📝 Commits (2)

c829280 Fix fatal error when not found dynamodb item
854d781 Change testTryingToFindAnItemThatDoesNotExist because return type of the getItem method e dont return null but a instance of Aws\Result and this in the get method return null

📊 Changes

2 files changed (+6 additions, -3 deletions)

View changed files

📝 lib/Doctrine/KeyValueStore/Storage/DynamoDbStorage.php (+2 -2)
📝 tests/Doctrine/Tests/KeyValueStore/Storage/DynamoDbTest.php (+4 -1)

📄 Description

No description provided

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/doctrine/KeyValueStore/pull/89 **Author:** [@gustavobatalha](https://github.com/gustavobatalha) **Created:** 4/17/2018 **Status:** ❌ Closed **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (2) - [`c829280`](https://github.com/doctrine/KeyValueStore/commit/c829280d315fac2a45cb7dd2d6d7ea0185a77db2) Fix fatal error when not found dynamodb item - [`854d781`](https://github.com/doctrine/KeyValueStore/commit/854d7817a79de4e93e6731fef13f4222cb850a97) Change testTryingToFindAnItemThatDoesNotExist because return type of the getItem method e dont return null but a instance of Aws\Result and this in the get method return null ### 📊 Changes **2 files changed** (+6 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `lib/Doctrine/KeyValueStore/Storage/DynamoDbStorage.php` (+2 -2) 📝 `tests/Doctrine/Tests/KeyValueStore/Storage/DynamoDbTest.php` (+4 -1) </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

admin added the pull-request label 2026-01-23 11:35:15 +01:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: doctrine/KeyValueStore#113