Spring Roo
  1. Spring Roo
  2. ROO-938

Introduce setter methods for Record objects

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Complete
    • Affects Version/s: None
    • Fix Version/s: 1.1.0.M3
    • Component/s: GWT
    • Labels:
      None

      Activity

      Hide
      Ray Ryan added a comment -

      RayC points out that implementing this as two separate classes (e.g. FooRecordImpl and FooRecordEditableImpl) will likely compile better.

      Show
      Ray Ryan added a comment - RayC points out that implementing this as two separate classes (e.g. FooRecordImpl and FooRecordEditableImpl) will likely compile better.
      Hide
      Ray Ryan added a comment -

      Could let us simplify deltavaluestore, e.g. the edited bean points straight to its record

      Show
      Ray Ryan added a comment - Could let us simplify deltavaluestore, e.g. the edited bean points straight to its record
      Hide
      Amit Manjhi added a comment -

      1d for generator, 1d for the remaining tasks.

      Show
      Amit Manjhi added a comment - 1d for generator, 1d for the remaining tasks.
      Hide
      Amit Manjhi added a comment -

      The basic version of this feature has been checked in: Roo commit e55daf7157606b4658388a6a6a01 and GWT commit http://code.google.com/p/google-web-toolkit/source/detail?r=8516

      The two optimizations requested above: DeltaValueStore simplification and trying to use two separate classes have not been done. This issue is no longer a blocker for M3 though. Moving it to M4.

      Show
      Amit Manjhi added a comment - The basic version of this feature has been checked in: Roo commit e55daf7157606b4658388a6a6a01 and GWT commit http://code.google.com/p/google-web-toolkit/source/detail?r=8516 The two optimizations requested above: DeltaValueStore simplification and trying to use two separate classes have not been done. This issue is no longer a blocker for M3 though. Moving it to M4.
      Hide
      Ray Ryan added a comment -

      Meh, those aren't important. This is fixed.

      Show
      Ray Ryan added a comment - Meh, those aren't important. This is fixed.

        People

        • Assignee:
          Amit Manjhi
          Reporter:
          Ray Ryan
        • Votes:
          0 Vote for this issue
          Watchers:
          0 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Time Tracking

            Estimated:
            Original Estimate - 2d
            2d
            Remaining:
            Remaining Estimate - 2d
            2d
            Logged:
            Time Spent - Not Specified
            Not Specified