[PR #1202] [MERGED] Check for null in quota-root. Some servers yield null values here #1335

New issue

Closed

opened 2026-02-25 21:37:58 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 21:37:58 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cypht-org/cypht/pull/1202
Author: @indridieinarsson
Created: 8/28/2024
Status: ✅ Merged
Merged: 8/30/2024
Merged by: @Baraka24

Base: master ← Head: quota_issue

📝 Commits (2)

7191ac6 Check for null in quota-root. Some servers yield null values here
53d3f98 Additionally check for existence of array key

📊 Changes

1 file changed (+9 additions, -7 deletions)

View changed files

📝 modules/imap/handler_modules.php (+9 -7)

📄 Description

Pullrequest

Issues

fixes #1201

Checklist

should be fairly straightforward, very unlikely to cause issues.

How2Test

Check that imap servers with user quotas are displayed correctly.
Check that imap servers without quotas, but for which quota check is still performed for some reasons, are bailed out silently, instead of logging an error message to the logs.

_{🔄 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/cypht-org/cypht/pull/1202 **Author:** [@indridieinarsson](https://github.com/indridieinarsson) **Created:** 8/28/2024 **Status:** ✅ Merged **Merged:** 8/30/2024 **Merged by:** [@Baraka24](https://github.com/Baraka24) **Base:** `master` ← **Head:** `quota_issue` --- ### 📝 Commits (2) - [`7191ac6`](https://github.com/cypht-org/cypht/commit/7191ac69d86c7221355f924128887b73ec27198b) Check for null in quota-root. Some servers yield null values here - [`53d3f98`](https://github.com/cypht-org/cypht/commit/53d3f98be284000b0199bb9383aaf572ab1fd46a) Additionally check for existence of array key ### 📊 Changes **1 file changed** (+9 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `modules/imap/handler_modules.php` (+9 -7) </details> ### 📄 Description ## Pullrequest  ### Issues - fixes #1201 ### Checklist should be fairly straightforward, very unlikely to cause issues. ### How2Test - [ ] Check that imap servers with user quotas are displayed correctly. - [ ] Check that imap servers without quotas, but for which quota check is still performed for some reasons, are bailed out silently, instead of logging an error message to the logs. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>