Uploaded image for project: 'Forge: Scalate'
  1. Forge: Scalate
  2. SCALATE-22

Switch the attribute defining syntax to be "@" ~ opt("import") ~ ("val"|"var" )

    Details

    • Type: Enhancement
    • Status: Resolved
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: 1.0
    • Fix Version/s: 1.0
    • Component/s: scalate-core
    • Labels:
      None

      Description

      For both ssp and scaml

        Gliffy Diagrams

          Activity

          Hide
          jastrachan James Strachan added a comment -

          BTW one possible downside is the attribute declaration will look like a normal bit of code...

          <%@ val foo : String = "blah" %>

          versus

          <% val foo : String = "blah" %>

          I guess the @ is the biggie.

          BTW if we declare a var, we should maybe make sure we call context.setAttribute("foo", foo) at the end of the template to output the new value back to the context

          Show
          jastrachan James Strachan added a comment - BTW one possible downside is the attribute declaration will look like a normal bit of code... <%@ val foo : String = "blah" %> versus <% val foo : String = "blah" %> I guess the @ is the biggie. BTW if we declare a var, we should maybe make sure we call context.setAttribute("foo", foo) at the end of the template to output the new value back to the context
          Hide
          chirino Hiram Chirino added a comment -

          yeah.. @ is the biggy.
          similar to how in ruby @foo and foo are 2 different variables. For me, that's ok.

          As for outputing the variable back.. I think that's a good idea. Will create a new issue for it.

          Show
          chirino Hiram Chirino added a comment - yeah.. @ is the biggy. similar to how in ruby @foo and foo are 2 different variables. For me, that's ok. As for outputing the variable back.. I think that's a good idea. Will create a new issue for it.
          Hide
          chirino Hiram Chirino added a comment -

          opened SCALATE-23

          Show
          chirino Hiram Chirino added a comment - opened SCALATE-23
          Hide
          chirino Hiram Chirino added a comment -

          the syntax change is now in.

          Show
          chirino Hiram Chirino added a comment - the syntax change is now in.

            People

            • Assignee:
              Unassigned
              Reporter:
              chirino Hiram Chirino
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development