Bug #43049

Lost page id after editing record out of Workspace Module

Added by Oliver Hader almost 7 years ago. Updated almost 2 years ago.

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

100%

TYPO3 Version:
4.5
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Given are those steps to reproduce:
  1. select a sub page in page tree
  2. open workspace module
  3. edit one of the workspace elements there
  4. save or close the editing process
  5. view is redirected to page id "0"

Encoding in the Javascript part to forward a correct returnUrl is wrong.
Instead of manually escaping sequences, encodeURIComponent can be used.

Associated revisions

Revision 301c0a79 (diff)
Added by Oliver Hader almost 7 years ago

[BUGFIX] Lost page id after editing record out of Workspace Module

Given are those steps to reproduce:
  • select a sub page in page tree
  • open workspace module
  • edit one of the workspace elements there
  • save or close the editing process
  • view is redirected to page id "0"

Encoding in the Javascript part to forward a correct returnUrl
is wrong. Instead of manually escaping sequences,
encodeURIComponent can be used.

Change-Id: Idda427015deb4595cc4e2c32542ba2d7a644e2b3
Fixes: #43049
Releases: 4.5, 4.6, 4.7, 6.0
Reviewed-on: http://review.typo3.org/16550
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader

Revision 4c467d08 (diff)
Added by Oliver Hader almost 7 years ago

[BUGFIX] Lost page id after editing record out of Workspace Module

Given are those steps to reproduce:
  • select a sub page in page tree
  • open workspace module
  • edit one of the workspace elements there
  • save or close the editing process
  • view is redirected to page id "0"

Encoding in the Javascript part to forward a correct returnUrl
is wrong. Instead of manually escaping sequences,
encodeURIComponent can be used.

Change-Id: Idda427015deb4595cc4e2c32542ba2d7a644e2b3
Fixes: #43049
Releases: 4.5, 4.6, 4.7, 6.0
Reviewed-on: http://review.typo3.org/16554
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader

Revision 4ea5cd99 (diff)
Added by Oliver Hader almost 7 years ago

[BUGFIX] Lost page id after editing record out of Workspace Module

Given are those steps to reproduce:
  • select a sub page in page tree
  • open workspace module
  • edit one of the workspace elements there
  • save or close the editing process
  • view is redirected to page id "0"

Encoding in the Javascript part to forward a correct returnUrl
is wrong. Instead of manually escaping sequences,
encodeURIComponent can be used.

Change-Id: Idda427015deb4595cc4e2c32542ba2d7a644e2b3
Fixes: #43049
Releases: 4.5, 4.6, 4.7, 6.0
Reviewed-on: http://review.typo3.org/16551
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader

Revision c6d601f2 (diff)
Added by Oliver Hader almost 7 years ago

[BUGFIX] Lost page id after editing record out of Workspace Module

Given are those steps to reproduce:
  • select a sub page in page tree
  • open workspace module
  • edit one of the workspace elements there
  • save or close the editing process
  • view is redirected to page id "0"

Encoding in the Javascript part to forward a correct returnUrl
is wrong. Instead of manually escaping sequences,
encodeURIComponent can be used.

Change-Id: Idda427015deb4595cc4e2c32542ba2d7a644e2b3
Fixes: #43049
Releases: 4.5, 4.6, 4.7, 6.0
Reviewed-on: http://review.typo3.org/16507
Reviewed-by: Marco Bresch
Tested-by: Marco Bresch
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader

History

#1 Updated by Gerrit Code Review almost 7 years ago

  • Status changed from Accepted to Under Review

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

#2 Updated by Gerrit Code Review almost 7 years ago

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

#3 Updated by Gerrit Code Review almost 7 years ago

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

#4 Updated by Gerrit Code Review almost 7 years ago

Patch set 1 for branch workspaces_4-7 has been pushed to the review server.
It is available at http://review.typo3.org/16554

#5 Updated by Oliver Hader almost 7 years ago

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

Applied in changeset commit:864803ca633168cff4e637d7039924927f96d08e.

#6 Updated by Michael Stucki almost 6 years ago

  • Category set to Workspaces

#7 Updated by Michael Stucki almost 6 years ago

  • Project changed from Workspaces & Versioning to TYPO3 Core
  • Category changed from Workspaces to Workspaces

#8 Updated by Riccardo De Contardi almost 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF