Java Parser: Problem with using Translation_details' Accreditation in ADL

Dear all,

the current java parser doesn't seem to allow changes to order of the
TRANSLATION_DETAILS (other than the order of language and author).

For example the following would fail:

translations = <
        ["de"] = <
            author = <
                ["organisation"] = <"my organisation">
                ["name"] = <"my name">
            >
            accreditation = <"my personal accreditation">
            language = <[ISO_639-1::de]>
        >

would fail, while the following would work

translations = <
        ["de"] = <
            author = <
                ["organisation"] = <"my organisation">
                ["name"] = <"my name">
            >
            language = <[ISO_639-1::de]>
            accreditation = <"my personal accreditation">
        >

I believe that changing the following in the adl.jj will fix this (ll.
733) - moved the ")*" further down to include not only authoor and
language, but also accreditation and otherDetails):

(
    <SYM_L_BRACKET> langAsKey = string_value() <SYM_R_BRACKET> <SYM_EQ>
<SYM_LT>
     
      (
        <SYM_LANG_LANGUAGE> <SYM_EQ> "<" language = code_phrase() ">"
      >
        <SYM_AUTHOR> <SYM_EQ> <SYM_LT>
          author = string_string_map()
        <SYM_GT>
      [
          <SYM_ACCREDITATION> <SYM_EQ> <SYM_LT>
            accreditation = string_value()
          <SYM_GT>
      ]
     
      [
        <SYM_LANG_OTHER_DETAILS> <SYM_EQ> <SYM_LT>
          otherDetails = string_string_map()
        <SYM_GT>
      ]
      )*
    <SYM_GT>
    {
      // TODO null terminology service
      td = new TranslationDetails(language, author, accreditation,
              otherDetails, terminologyService);
      map.put(langAsKey, td);
    }
  )+

Rong and others, do you agree?

Cheers
Sebastian

Sebastian,

I think the fix should work. Please go ahead and commit the change. It
will be great if you can attach a testcase too.

Cheers,
Rong