Project

General

Profile

Actions

Bug #70792

closed

System Information Panel: Impossible to extend

Added by Patrick Schriner about 9 years ago. Updated about 9 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2015-10-16
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

Contrary to the documentation given in

https://wiki.typo3.org/TYPO3.CMS/Releases/7.2/Feature#Feature:_.2365767_-
_System_Information_Dropdown

the code in "SystemInformationToolbarItem" expects only a single "item" return for both the "loadMessages" and the "getSystemInformation" signal, due to not passing "$this" as an argument and there not being any API to actually add messages // system information. This is an issue because the loadMessages Signal is already getting registered by the system "belog" extension.

Furthermore if there were two methods

(e.g.) "addSystemInformation" and "addMessage" in the class "SystemInformationToolbarItem"

you could legitimately remove one of the signals - as they are identical otherwise.


Related issues 1 (0 open1 closed)

Is duplicate of TYPO3 Core - Bug #70567: System Information Panel: Impossible to extendClosedPatrick Schriner2015-10-10

Actions
Actions #1

Updated by Mathias Schreiber about 9 years ago

  • Category set to Backend User Interface
  • Status changed from New to Needs Feedback
  • Assignee set to Mathias Schreiber
  • Target version set to 7 LTS

duplicate?

Actions #2

Updated by Gerrit Code Review about 9 years ago

  • Status changed from Needs Feedback to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/44109

Actions #3

Updated by Patrick Schriner about 9 years ago

Mathias Schreiber wrote:

duplicate?

On purpose. Just two variants trying to fix the same issue. Both bad on a different level.

I don't mind how this bug is solved, I just don't want to have the broken current state for all 7 LTS eternity

Test case: https://github.com/pschriner/additional_sysinfo

Actions #4

Updated by Markus Klein about 9 years ago

  • Category deleted (Backend User Interface)
  • Status changed from Under Review to Closed
  • Assignee deleted (Mathias Schreiber)
  • Target version deleted (7 LTS)

Closing this as duplicate, the patch is already abandoned.

Actions

Also available in: Atom PDF