Bug #20598

Error after hide/show element

Added by Yannick Pavard over 12 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Workspaces
Target version:
-
Start date:
2009-06-08
Due date:
% Done:

100%

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

Description

Hi,

This is my error :

1: Could not be edited in offline workspace in the branch where found (failure state: 'Online record was not in versionized branch!'). Auto-creation of version not allowed in workspace!

when I use hide/show button in list mode.

The first time is correct, but error appear after other click on this button.

Can you patch it ?

sorry for my bad english ;)

(issue imported from #M11303)


Files

class.db_list_extra.patch (1.18 KB) class.db_list_extra.patch Administrator Admin, 2009-11-26 21:11
0011303_hide_show.diff (1.77 KB) 0011303_hide_show.diff Administrator Admin, 2010-05-05 16:06
0011303_hide_show_4.4.4.diff (1.69 KB) 0011303_hide_show_4.4.4.diff Administrator Admin, 2010-10-08 15:12

Related issues

Precedes TYPO3 Core - Bug #32780: Errors when hiding/unhiding and moving contentClosed2011-12-22

Actions
#1

Updated by Julia Pardanaud about 12 years ago

Hi,
When you use hide/unhide button on a versionized record, it's like if TYPO3 tries to change the LIVE version instead : it's the uid of the LIVE version which is sent.

I propose this patch which change this id when it's necessary (version 4.3).
Instead of changing class.db_list_extra.inc, we could try to change class.t3lib_tcemain.php. What is the best solution ?

sorry for my bad english too ;)

#2

Updated by Robert Heel over 11 years ago

This issue affects the info, move up/down, hide, delete and cut buttons ...
Only on the create translation flags it's correct to use the original id.

#3

Updated by Robert Heel over 11 years ago

Seems to be intentional that move and cut creates a new type of version element.
Added a easier patch 0011303_hide_show.diff

#4

Updated by Benni Mack about 11 years ago

Please check if this issue still exists with trunk or 4.4.2, should be fixed since 4.4.1.

#5

Updated by Robert Heel about 11 years ago

Didn't work with 4.4.2. When clicking two times on the show/hide button in workspace list view, this error message appears:
1: [newlog()] Could not be edited in offline workspace in the branch where found (failure state: 'Online record was not in versionized branch!'). Auto-creation of version not allowed in workspace!

#6

Updated by Simon Schaufelberger almost 11 years ago

still happens in 4.4.6 with templavoila and the light bulb. with the context menu it works

#7

Updated by Francois Suter almost 11 years ago

Indeed the bug still exists across all branches and trunk.

For TemplaVoilà, I reported the issue as a separate bug report.

#8

Updated by Sonja Schubert over 10 years ago

  • Status changed from New to Closed
  • Target version deleted (0)
#9

Updated by Oliver Hader over 10 years ago

Since currently only element versioning is supported, this issues was set to be closed

#10

Updated by Robert Heel over 10 years ago

Issue still exists in 4.5.2 - Why closed?

#11

Updated by Maximilian Schmid over 10 years ago

Issue still exists in 4.5.3 - 0011303_hide_show.diff solves this issue. Please include it in the next release (4.5.4)!

#12

Updated by Tobias Liebig over 10 years ago

Can confirm, the issue still exists in 4.5.2.
I'll test the patch

#13

Updated by Oliver Tempel over 10 years ago

Can confirm, too. Version 4.5.2 is still buggy.
No autopatch.

Used diff 4.4.4 to patch manually.

Added line @ 1138 $ver_uid=$row['_ORIG_uid'] ? $row['_ORIG_uid'] : $row['uid'];
Changed line @1274 $params='&data['.$table.']['.$ver_uid.']['.$hiddenField.']=0';
Changed line @1281 $params='&data['.$table.']['.$ver_uid.']['.$hiddenField.']=1';

What about up/down patch, or the other $row['uid'] in these function ?

#14

Updated by Michael McManus over 10 years ago

Oliver Hader wrote:

Since currently only element versioning is supported, this issues was set to be closed

Hi Oliver,

Thanks for your feedback here and your work on the project, in general. I'm having a hard time understanding why this bug is no longer open, based your explanation re: element versioning. How is it that the Show/Hide icon in the element context menu is tied to the proper versioned uid, but not with the icon that appears in the list?

Thanks!

-Michael

#15

Updated by Tolleiv Nietsch over 10 years ago

  • Status changed from Closed to Accepted

Agree - still valid

#16

Updated by Mr. Hudson about 10 years ago

Patch set 1 of change I43ceaf2443dff7c3d666811726353f3b17b81182 has been pushed to the review server.
It is available at http://review.typo3.org/5633

#17

Updated by Mr. Hudson about 10 years ago

Patch set 2 of change I43ceaf2443dff7c3d666811726353f3b17b81182 has been pushed to the review server.
It is available at http://review.typo3.org/5633

#18

Updated by Simon Schaufelberger about 10 years ago

  • Status changed from Accepted to Resolved
  • % Done changed from 0 to 100
#19

Updated by Bart Dubelaar about 10 years ago

I am still experiencing this bug in 4.5.8. At least hiding and unhiding a page will throw this error, and moving a content element to the top position on a page will throw this error too. Is the merged patch already in 4.5.8 or is the target 4.5.9?

#20

Updated by Bart Dubelaar about 10 years ago

Ok, this patch is merged in 4.5.8 but the issue is not resolved. Please reopen this ticket.

#21

Updated by Bart Dubelaar almost 10 years ago

I am currently working on improvement of the patch on class.db_list_extra.inc. However, I think that the current approach is only fixing the symptoms and not the cause.
The actual cause is in the process_datamap function of class.t3lib_tcemain.php. This function should detect if there is an existing version available in the current workspace, which it currently does not.

The process_cmdmap function does detect this. This discrepancy becomes very clear now that I am also trying to fix repositioning of an element. This action needs a cmdmap and a datamap (for the colpos) where the cmdmap takes the live id and the datamap takes the versioned id...

I can finish the patches that fix the symptoms, however I would like to have some advice on tackling the cause in tcemain. I can also work on a patch for tcemain, but this is at bit trickier and I don't know if my analysis of the problem is 100% correct.

#22

Updated by Michael Stucki almost 8 years ago

  • Category set to Workspaces
#23

Updated by Michael Stucki almost 8 years ago

  • Project changed from 624 to TYPO3 Core
  • Category changed from Workspaces to Workspaces
#24

Updated by Benni Mack about 3 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF