Bug #2255
Sunstone template editing in wizzard mode destroys RAW section
Status: | Closed | Start date: | 08/01/2013 | |
---|---|---|---|---|
Priority: | High | Due date: | ||
Assignee: | Daniel Molina | % Done: | 100% | |
Category: | Sunstone | |||
Target version: | Release 4.4 | |||
Resolution: | fixed | Pull request: | ||
Affected Versions: | OpenNebula 4.2 |
Description
Sunstone template editing in wizzard mode destroys RAW section, and it's very sad due to this: http://dev.opennebula.org/issues/2143
Steps to reproduce:
1. Create template in Advanced mode with simple RAW section
2. Open template for update in wizard mode
3. Change anything
4. Click update
5. Open template for update in advanced mode: there's no RAW section
I created fast'n'dirty patch which adds RAW tab, it's in attachment (sorry for my JS, i use it for the first time today :))
Related issues
Associated revisions
bug #2255: Add RAW section to template creation form
bug #2255: Add RAW tab to views conf
bug #2255: Update css
bug #2255: Hide data_vmx by default
bug #2255: Add custom tags to template creation form
bug #2255: Add RAW section to template creation form(cherry picked from commit fc32d730158c64bc7e7e429703cd472385d33379)
bug #2255: Add RAW tab to views conf(cherry picked from commit b3c0f6a8525eba1e322025e657bd3a1657ef0802)
bug #2255: Update css(cherry picked from commit 61558708ed12ea2000c5a265a3752284794862cf)
bug #2255: Hide data_vmx by default(cherry picked from commit b6c75a34d12a3acf364c812b0026be220b0d49a2)
bug #2255: Add custom tags to template creation form(cherry picked from commit 97b713fd7f0f386898f4f3c07010a8d44db531fe)
bug #2255: Escape double quotes provided in an input text in Sunstone template wizard
bug #2255: Fix bug when a tag contains double quotes in the user template info panel
bug #2255: Fix bug when a tag contains double quotes in the user template info panel
(cherry picked from commit 3f6716bb0308d616f8ef458d9793760c5dd6dcc7)
bug #2255: Escape double quotes provided in an input text in Sunstone template wizard
(cherry picked from commit b501d34fc3ea8145ef83436586f471877d606b63)
History
#1 Updated by Alexey Medvedchikov almost 8 years ago
- File opennebula.diff added
#2 Updated by Daniel Molina almost 8 years ago
- Status changed from Pending to New
- Assignee set to Daniel Molina
- Priority changed from Low to High
- Target version set to Release 4.4
#3 Updated by Daniel Molina almost 8 years ago
- Status changed from New to Closed
- % Done changed from 0 to 100
- Resolution set to fixed
Thank you for the patch. I have just included it in the repo. I have also included a "custom tags" section in case additional parameters are defined in the vm template.
#4 Updated by Alexey Medvedchikov almost 8 years ago
Thank you. Your changeset works fine for me.
#5 Updated by Laurent Grawet almost 8 years ago
Hi,
There is a problem with function addslashes( str ) in sunstone/public/js/plugins/templates-tab.js:
it doubles backslashes on each update.
#6 Updated by Laurent Grawet almost 8 years ago
About the addslashes(str) problem:
- The regex shouldn't add backslash before single quote char because OpenNebula doesn't need it and htmlDecode() doesn't strip it on read.
- The template update should fail if the field contains a backslash character. There is also the htmlDecode() problem
- Double quotes should be replaced with single quotes.
Then you will have the same behavior as command line tool.
#7 Updated by Ruben S. Montero almost 8 years ago
- Status changed from Closed to New
#8 Updated by Ruben S. Montero almost 8 years ago
- Resolution deleted (
fixed)
#9 Updated by Daniel Molina over 7 years ago
Hi Laurent and Alexey,
I have uploaded a commit that should fix these issues. The commit is already in the master and one-4.4 branches
#10 Updated by Daniel Molina over 7 years ago
- Status changed from New to Closed
- Resolution set to fixed