[PR #706] [MERGED] Removed substr call in amqp routing key generator #1307

New issue

Closed

opened 2026-03-04 03:09:15 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 03:09:15 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Seldaek/monolog/pull/706
Author: @acrobat
Created: 12/20/2015
Status: ✅ Merged
Merged: 4/15/2016
Merged by: @Seldaek

Base: master ← Head: amqp_handler_fix

📝 Commits (1)

dbccc56 Removed substr call in amqp routing key generator

📊 Changes

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

View changed files

📝 src/Monolog/Handler/AmqpHandler.php (+1 -6)
📝 tests/Monolog/Handler/AmqpHandlerTest.php (+2 -2)

📄 Description

Fixed todo from #197

Remove substr TODO in AmqpHandler

_{🔄 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/Seldaek/monolog/pull/706 **Author:** [@acrobat](https://github.com/acrobat) **Created:** 12/20/2015 **Status:** ✅ Merged **Merged:** 4/15/2016 **Merged by:** [@Seldaek](https://github.com/Seldaek) **Base:** `master` ← **Head:** `amqp_handler_fix` --- ### 📝 Commits (1) - [`dbccc56`](https://github.com/Seldaek/monolog/commit/dbccc56a442fcb760050a0e527ce03f73446c639) Removed substr call in amqp routing key generator ### 📊 Changes **2 files changed** (+3 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `src/Monolog/Handler/AmqpHandler.php` (+1 -6) 📝 `tests/Monolog/Handler/AmqpHandlerTest.php` (+2 -2) </details> ### 📄 Description Fixed todo from #197 > Remove substr TODO in AmqpHandler --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>