Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: Trunk
    • Fix Version/s: 16.11.01
    • Component/s: ALL APPLICATIONS
    • Labels:
      None

      Description

      This is a placeholder issue to capture related issues to regarding replacing existing beanshell code with Groovy code. It helps planning and communicating.

        Activity

        Hide
        jacques.le.roux Jacques Le Roux added a comment - - edited

        Great, BTW when changing from BSH to groovy it would have been better to change from

        &

        (and such) to & (and such).
        But no worries that can be done automatically with a gloabl S/R in Eclipse after this umbrella has been closed (ie all sub-tasks done)

        Show
        jacques.le.roux Jacques Le Roux added a comment - - edited Great, BTW when changing from BSH to groovy it would have been better to change from & (and such) to & (and such). But no worries that can be done automatically with a gloabl S/R in Eclipse after this umbrella has been closed (ie all sub-tasks done)
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Finally since there are cases where you need to be careful and cases with multi-lines changes I think it's better to do it "by hand" (ie local S/R)

        Show
        jacques.le.roux Jacques Le Roux added a comment - Finally since there are cases where you need to be careful and cases with multi-lines changes I think it's better to do it "by hand" (ie local S/R)
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        I'd do a "${bsh" -> "${groovy" in *.xml then S/R with some care on where to apply et voilĂ . I'd do that but I'd then prefer to also improve the code, else I don't see the point...

        Show
        jacques.le.roux Jacques Le Roux added a comment - I'd do a "${bsh" -> "${groovy" in *.xml then S/R with some care on where to apply et voilĂ . I'd do that but I'd then prefer to also improve the code, else I don't see the point...
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Done at r1682118

        Show
        jacques.le.roux Jacques Le Roux added a comment - Done at r1682118

          People

          • Assignee:
            pfm.smits Pierre Smits
            Reporter:
            pfm.smits Pierre Smits
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development