Issue Details (XML | Word | Printable)

Key: GLASSFISH-12418
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Critical Critical
Assignee: kenpaulsen
Reporter: Anissa Lam
Votes: 0
Watchers: 4
Operations

If you were logged in you would be able to see more operations.
glassfish

convert all property table related handlers to REST

Created: 29/Jun/10 02:12 PM   Updated: 26/Nov/10 12:14 AM   Resolved: 27/Sep/10 03:10 PM
Component/s: admin_gui
Affects Version/s: 3.1
Fix Version/s: 3.1_ms04

Time Tracking:
Not Specified

Environment:

Operating System: All
Platform: All

Issue Links:
Dependency

Issuezilla Id: 12,418
Tags:
Participants: Anissa Lam, Jason Lee and kenpaulsen


 Description  « Hide

Now that we can use REST API to create/delete/get/set property values, (issue#
12156 has been fixed), we need to finishing up the conversion of the property
table.

Requesting Jason to convert those handlers thats related to property table from
using AMX to REST.
Here is the handlers that needs the work.

setProxyProperties()
updateProxyProperties()
propMapToList()
getProxyProperties()



Anissa Lam added a comment - 06/Jul/10 04:34 PM

MS3.
Many pages conversion cannot be fully completed because it is waiting for this.


Jason Lee added a comment - 12/Jul/10 06:28 PM

Adding blocking issue


Jason Lee added a comment - 13/Jul/10 07:05 AM

Starting...


kenpaulsen added a comment - 12/Aug/10 10:45 AM

Added support for using REST to display properties. To take advantage of this
include "common/shared/propertyDescTable.inc" and set the request scoped
"tableList" attribute populated as a List<Map<String,String>> with values for
"name", "value", and "description" (optional). You DO NOT need to pre-
populate values for "selected", or "encodedName" – this will be done for you.

For an example, take a look at this file:

common/src/main/resources/appServer/serverInstAppsConfig.jsf

Please report any problem using this code to this Issue Tracker number.


kenpaulsen added a comment - 18/Aug/10 01:52 PM

Persisting properties was broken. I have fixed this. A couple notes:

  • The name/value/description (backend properties) must be in their own
    List<Map> called tableList. This is typically obtainable directly from a REST
    request and then extracted:

gf.restRequest(
endpoint="#{pageSession.selfUrl}/property.json"
method="GET"
result="#{requestScope.propTable}");
setPageSessionAttribute(
key="tableList"
value="#{requestScope.propTable.data.extraProperties.properties}");

  • This List<Map> must be set in a pageSession attribute named "tableList"
  • Finally #include "/common/shared/propertyDescTable.inc" should be used to
    show the properties table. This is not a change – it should have always been
    like this.
  • Ken

Anissa Lam added a comment - 27/Sep/10 03:10 PM


We have been able to set Property through REST for sometime. closing the bug.