actually the principle is OK; so remove the todo, but:
1) why record an update? Call service.setProp() (which will record an update)
2) [doc]
constants for node props should be used. They should stay in a class: NodePropertiesConstants, placed in the root package of the plugin. Both Java and Flex.