Bug #11511

For the top menu a property "typeMenu" is defined, but never used

Added by Chris topher over 2 years ago. Updated over 2 years ago.

Status:New Start date:2010-12-16
Priority:Should have Due date:
Assignee:- % Done:

0%

Category:-
Target version:-
Votes: 0

Description

It is about a property and a value:

  • In ext_localconf around line 50 the property menuBar.typeMenu is set to the value "text, header, image, html". Seems like this should restrict the types of content elements in the top menu.
    But typeMenu is never used in the extension.
  • In view/class.tx_feeditadvanced_adminpanel.php around line 266 menuBar.config (defines which blocks of options to show in the top menu) is set to a string containing "type".
    I think this values was thought to set the visibility of typeMenu, because the value "context" here sets the visibility of contextMenu (which is another part in the top menu).
    But it seems like the value "type" is never used.

Dead code?!
=> Remove?


Related issues

related to Advanced Frontend Editing - Bug #11510: "imageType" is set in ext_localconf, but never used New 2010-12-16

History

Updated by Steffen Kamper over 2 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Updated by Chris topher over 2 years ago

  • Status changed from Resolved to New
  • % Done changed from 100 to 0

Also available in: Atom PDF