Project

General

Profile

Actions

Bug #42906

closed

cshmanual is a required extension?

Added by Eric Chavaillaz about 12 years ago. Updated almost 10 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2012-11-13
Due date:
% Done:

0%

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

Description

Hi,

Why did we cannot remove cshmanual?

Thanks,
Eric


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #40665: CSH Manual Popup brokenClosedJigal van Hemert2012-09-06

Actions
Actions #1

Updated by Christian Kuhn about 12 years ago

  • Project changed from 2269 to TYPO3 Core
Actions #2

Updated by Christian Kuhn about 12 years ago

The cshmanual was made required in #40665 because the csh popups throw exceptions if the extension is not loaded.

Background: The script for the manual popups was typo3/view_help.php in 4.7 and was moved to ext:cshmanual in order to reduce the number of entry points (it now goes through mod.php).
Funnily, the main ? symbols still work with basic information, only the csh manual popup is broken if the extension is removed from REQUIRED_EXTENSIONS list.

I think, it should be possible to unload the extension again. Imho, the popup link should then just not be rendered.

If someone wants to fix that: At first glance it looks like as if the according .js script is t3lib/js/extjs/notifications.js ... this might need some adaption.

Actions #3

Updated by Eric Chavaillaz about 12 years ago

Ok, I have not seen the #40665...

But for me this extension is not a required extension because the system is not depending on it.

It would be great if this extension can be unloaded ans then the popup link is not rendered.

Thanks Christian for the explanation!

Actions #4

Updated by Eric Chavaillaz over 11 years ago

In fact the extension "cshmanual" must be required only if "context_help" is loaded... True?

Thanks.

Actions #5

Updated by Mathias Schreiber almost 10 years ago

  • Status changed from New to Closed
  • Is Regression set to No

fixed on master

Actions

Also available in: Atom PDF