Bug #38705

Bug #46389: Faulty Workspace Placeholders

tslib_menu::makeMenu - Move placeholder are displayed in navigation during WS preview

Added by Michael Klapper about 3 years ago. Updated over 2 years ago.

Status:Resolved Start date:2012-07-06
Priority:Should have Due date:
Assigned To:- % Done:

100%

Category:- Spent time: -
Target version:-
TYPO3 Version:4.7 Is Regression:
PHP Version: Sprint Focus:
Complexity:

Description

Live WS
Initial page tree situation:

+---[1] First
+---[2] Second
+---[3] Third

Displayed navigation in frontend

First |  Second | Third

Custom created WS
Create a new page "[4] Fourth" and move the page "[3] Third" as seen below:

+---[1] First
+---[4] Fourth
¦   +---[3] Third
+---[2] Second

If you now open the WS preview you will notice that the navigation shows something like:

First | Fourth | Second | Third

Comment
I would expect that the move placeholder of page "[3] Third" is not visible in the first navigation level any more.

Quick navigation snipp to reproduce:

navigation = COA
navigation {
    10 = HMENU
    10 {
        entryLevel = 0
        special = directory
        special.value = __YOUR_ENTRY_POINT__

        1 = TMENU
        1 {
            NO {
            }
        }
    }
}

Associated revisions

Revision 6a472ab1
Added by Benjamin Mack over 2 years ago

[BUGFIX] Hide move placeholder in WS preview

Move placeholder are visible in the navigation
during WS preview if they are moved for example
from the first to the second level.

Change-Id: I7e3339488d9d64cdf04154fb0334880f76576f8d
Fixes: #38705
Releases: 6.0,4.7,4.6,4.5
Reviewed-on: https://review.typo3.org/12666
Reviewed-by: Benjamin Mack
Tested-by: Benjamin Mack

Revision 1b284b78
Added by Benjamin Mack over 2 years ago

[BUGFIX] Hide move placeholder in WS preview

Move placeholder are visible in the navigation
during WS preview if they are moved for example
from the first to the second level.

Change-Id: I7e3339488d9d64cdf04154fb0334880f76576f8d
Fixes: #38705
Releases: 6.0,4.7,4.6,4.5
Reviewed-on: https://review.typo3.org/12666
Reviewed-by: Benjamin Mack
Tested-by: Benjamin Mack
Reviewed-on: https://review.typo3.org/19711

Revision 23fbac94
Added by Benjamin Mack over 2 years ago

[BUGFIX] Hide move placeholder in WS preview

Move placeholder are visible in the navigation
during WS preview if they are moved for example
from the first to the second level.

Fixes: #38705
Releases: 6.1,6.0,4.7,4.5
Change-Id: I7bdeee0246bb34bdc5b8baf963fad6b262078877
Reviewed-on: https://review.typo3.org/19712
Reviewed-by: Benjamin Mack
Tested-by: Benjamin Mack

Revision c01f4ee6
Added by Benjamin Mack over 2 years ago

[BUGFIX] Hide move placeholder in WS preview

Move placeholder are visible in the navigation
during WS preview if they are moved for example
from the first to the second level.

Fixes: #38705
Releases: 6.1,6.0,4.7,4.5
Change-Id: I7bdeee0246bb34bdc5b8baf963fad6b262078877
Reviewed-on: https://review.typo3.org/19712
Reviewed-by: Benjamin Mack
Tested-by: Benjamin Mack
Reviewed-on: https://review.typo3.org/19713

History

#1 Updated by Gerrit Code Review about 3 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12666

#2 Updated by Gerrit Code Review about 3 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12666

#3 Updated by Oliver Hader over 2 years ago

  • Parent task set to #46389

#4 Updated by Gerrit Code Review over 2 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/12666

#5 Updated by Gerrit Code Review over 2 years ago

Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/19711

#6 Updated by Gerrit Code Review over 2 years ago

Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/19712

#7 Updated by Gerrit Code Review over 2 years ago

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at https://review.typo3.org/19713

#8 Updated by Benjamin Mack over 2 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF