Documentation
  1. Documentation
  2. DOCS-44

BONUS:POSTMOVEADD cannot lower speed

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.17.5
    • Component/s: None
    • Labels:
      None

      Description

      As noted in the comments, the BONUS:POSTMOVEADD tag is coded to not be able to lower the base speed. Could you add a note to the docs entry on this tag please?

      --------------------------------------------------
      Reported on List File Help

      BONUS:POSTMOVEADD|TYPE.Walk|-10 -> Base speed does not change.

      Could I set it to an exact value of 20 regardless of any other circumstances?

      I'll investigate.

        Activity

        Hide
        James Dempsey
        added a comment -

        Have you tried changing Walk to WALK?

        Show
        James Dempsey
        added a comment - Have you tried changing Walk to WALK?
        Hide
        James Dempsey
        added a comment -

        ok, tracing this through, it is coded up explicitly not to lower speed with a POSTMOVEADD:
        // because POSTMOVE is magical movement which should not be
        // multiplied by magical items, etc, we now see which is larger,
        // (baseMove + postMove) or (baseMove * moveMultiplier)
        // and keep the larger one, discarding the other
        Looks like we should be updating the docs.

        Show
        James Dempsey
        added a comment - ok, tracing this through, it is coded up explicitly not to lower speed with a POSTMOVEADD: // because POSTMOVE is magical movement which should not be // multiplied by magical items, etc, we now see which is larger, // (baseMove + postMove) or (baseMove * moveMultiplier) // and keep the larger one, discarding the other Looks like we should be updating the docs.
        James Dempsey
        made changes -
        Field Original Value New Value
        Summary BONUS:POSTMOVEADD|TYPE.Walk|-10 -> Base speed does not change. BONUS:POSTMOVEADD cannot lower speed
        Description Reported on List File Help

        BONUS:POSTMOVEADD|TYPE.Walk|-10 -> Base speed does not change.

         Could I set it to an exact value of 20 regardless of any other circumstances?

        I'll investigate.
        As noted in the comments, the BONUS:POSTMOVEADD tag is coded to not be able to lower the base speed. Could you add a note to the docs entry on this tag please?

        --------------------------------------------------
        Reported on List File Help

        BONUS:POSTMOVEADD|TYPE.Walk|-10 -> Base speed does not change.

         Could I set it to an exact value of 20 regardless of any other circumstances?

        I'll investigate.
        James Dempsey
        made changes -
        Project Code [ 10020 ] Documentation [ 10023 ]
        Key CODE-240 DOCS-44
        Workflow PCGen Code [ 10834 ] jira [ 10840 ]
        Andrew Maitland
        made changes -
        Assignee Andrew Maitland [ legacyking ]
        Hide
        Eric C. Smith
        added a comment -

        svn 14245

        Show
        Eric C. Smith
        added a comment - svn 14245
        Hide
        Eric C. Smith
        added a comment -

        svn 14245

        Show
        Eric C. Smith
        added a comment - svn 14245
        Eric C. Smith
        made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 5.17.5 [ 10133 ]
        Resolution Fixed [ 1 ]
        Eric C. Smith
        made changes -
        Assignee Eric C. Smith [ maredudd ]
        Hide
        Eric C. Smith
        added a comment -

        Added note on interaction between POSTMOVEADD and MOVEMULT to both BONUS tag entries.
        svn 15178

        Show
        Eric C. Smith
        added a comment - Added note on interaction between POSTMOVEADD and MOVEMULT to both BONUS tag entries. svn 15178
        Eric C. Smith
        made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Eric C. Smith
            Reporter:
            Andrew Maitland
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: