ShieldMeta

XMLWordPrintable

    • Type: New Feature
    • Resolution: Done
    • Priority: Minor
    • None
    • Affects Version/s: None

      As we know, you can change the pattern of a Shield.
      (Craft a banner and a shield together; http://i.imgur.com/H5Wn1NU.png)
      However, this is not available on Spigot. We should handle ShieldMeta just like BannerMeta, OR, apply BannerMeta to Shields.

      Thanks.

          [SPIGOT-1765] ShieldMeta

          md_5 added a comment -

          That's not how shields work though They are literally the combination of a shield and a banner. Pretending a shield is a banner when they are two combined things is silly.

          md_5 added a comment - That's not how shields work though They are literally the combination of a shield and a banner. Pretending a shield is a banner when they are two combined things is silly.

          Robi F. added a comment -

          Honestly, I don't find that really smart. It'd be easier to simply use a ShieldMeta, or BannerMeta (for shields, of course.)

          Robi F. added a comment - Honestly, I don't find that really smart. It'd be easier to simply use a ShieldMeta, or BannerMeta (for shields, of course.)

          md_5 added a comment -

          As I said above and as the stack trace says, it's a BlockStateMeta. So use that and set the state to a banner.

          md_5 added a comment - As I said above and as the stack trace says, it's a BlockStateMeta. So use that and set the state to a banner.

          Robi F. added a comment - - edited

          It gave me a stacktrace, but I will try this again.

          (P.S: I deleted my comment because I wasn't sure that it was supposed to work, or it's just got added now.)

          Edit:
          http://pastebin.com/KsK13wpM
          http://pastebin.com/ez0UCKS9
          And yes, I am running the latest version.

          Robi F. added a comment - - edited It gave me a stacktrace, but I will try this again. (P.S: I deleted my comment because I wasn't sure that it was supposed to work, or it's just got added now.) Edit: http://pastebin.com/KsK13wpM http://pastebin.com/ez0UCKS9 And yes, I am running the latest version.

          md_5 added a comment -

          What about it didnt work? Its a blockstate meta, you attach the ban er as the state.

          md_5 added a comment - What about it didnt work? Its a blockstate meta, you attach the ban er as the state.

          md_5 added a comment -

          Banner meta is applicable to shields.

          md_5 added a comment - Banner meta is applicable to shields.

            Assignee:
            Unassigned
            Reporter:
            Robi F.
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: