sIKE23 / Mage-Wars

Mage Wars for OCTGN
7 stars 5 forks source link

Add forum code (BBCode) export to Deckbuilder #4

Closed lettucemode closed 10 years ago

lettucemode commented 10 years ago

The official forums support showing card previews. It would be cool if there were an option to export decks from the OCTGN deckbuilder in such a way that when pasted into a forum post, the cards showed previews. The card IDs are already in the OCTGN database.

sIKE23 commented 10 years ago

I have been working with Pixelgeek to have the AW forum export into the OCTGN format.

sIKE23 commented 10 years ago

I mis-understood this, I like it a lot! I was think that you were wanting to export into sbb format....

sIKE23 commented 10 years ago

Here is the fourm code output for my Priest deck:

[spellbook] [spellbookheader] [spellbookname]Holy Roller[/spellbookname] [mage]Priest[/mage] [/spellbookheader] [spells] [spellclass]Attack[/spellclass] [mwcard=FWA02]1 x Force Hammer[/mwcard] [mwcard=FWA04]1 x Hurl Boulder[/mwcard] [mwcard=mw1a09]1 x Jet Stream[/mwcard] [mwcard=MWSTX1CKA01]1 x Surging Wave[/mwcard] [spellclass]Conjuration[/spellclass] [mwcard=mw1j08]1 x Hand of Bim-Shalla[/mwcard] [mwcard=mw1j12]2 x Mana Crystal[/mwcard] [mwcard=FWJ08]1 x Renewing Spring[/mwcard] [mwcard=DNW01]1 x Bloodspine Wall[/mwcard] [spellclass]Creature[/spellclass] [mwcard=MWSTX1CKC06]1 x Guardian Angel[/mwcard] [mwcard=mw1c20]1 x Highland Unicorn[/mwcard] [mwcard=mw1c22]2 x Knight of Westlock[/mwcard] [spellclass]Enchantment[/spellclass] [mwcard=MWSTX1CKE01]1 x Armor Ward[/mwcard] [mwcard=mw1e01]3 x Bear Strength[/mwcard] [mwcard=mw1e02]1 x Block[/mwcard] [mwcard=mw1e05]1 x Cheetah Speed[/mwcard] [mwcard=mw1e11]1 x Divine Intervention[/mwcard] [mwcard=mw1e28]2 x Mongoose Agility[/mwcard] [mwcard=mw1e29]2 x Nullify[/mwcard] [mwcard=mw1e34]1 x Reverse Attack[/mwcard] [mwcard=mw1e35]2 x Reverse Magic[/mwcard] [mwcard=mw1e36]1 x Rhino Hide[/mwcard] [spellclass]Equipment[/spellclass] [mwcard=mw1q27]1 x Dawnbreaker Ring[/mwcard] [mwcard=mw1q06]1 x Dragonscale Hauberk[/mwcard] [mwcard=mw1q07]1 x Elemental Cloak[/mwcard] [mwcard=mw1q08]1 x Elemental Wand[/mwcard] [mwcard=mw1q19]1 x Mage Wand[/mwcard] [mwcard=mw1q23]1 x Regrowth Belt[/mwcard] [mwcard=FWQ10]1 x Storm Drake Hide[/mwcard] [mwcard=MWSTX1CKQ01]1 x Sunfire Amulet[/mwcard] [mwcard=mw1q29]1 x Staff of Asyra[/mwcard] [spellclass]Incantation[/spellclass] [mwcard=mw1i06]3 x Dispel[/mwcard] [mwcard=mw1i07]2 x Dissolve[/mwcard] [mwcard=mw1i12]1 x Force Push[/mwcard] [mwcard=mw1i17]1 x Minor Heal[/mwcard] [mwcard=mw1i24]2 x Seeking Dispel[/mwcard] [mwcard=mw1i28]2 x Teleport[/mwcard] [mwcard=mw1i19]1 x Piercing Strike[/mwcard] [/spells] [cost]Total cost: 120 pts[/cost] [/spellbook]

lettucemode commented 10 years ago

This appears to work...lemme know what you think. This wouldn't be possible if you hadn't taken the time to include the card IDs in the .xml files, so kudos to you for that!

sIKE23 commented 10 years ago

Looks really good! Should we say anything about Promo's not be support on the AW website? Other than that stray thought we can place a bow on it and call it done.

sIKE23 commented 10 years ago

Spoke to soon, the validator is now calculating spellpoints wrong. Use the Beastmaster (Johktari) as my reference deck and it is now calculating out at 195 spellpoints.

sIKE23 commented 10 years ago

This might be the result of my work though with the new Card properties....

lettucemode commented 10 years ago

That's a good catch with the promos, I'll put that in.

I forgot about the changed card properties. Updating the validators for those is on our March list as #27.

sIKE23 commented 10 years ago

Maybe add (built by OCTGN SBB) to the title?

sIKE23 commented 10 years ago

In my new post I manually added in the above reference so you could see what I was talking about

lettucemode commented 10 years ago

This way looks a bit cleaner to me, what do you think? untitled

sIKE23 commented 10 years ago

Perfect!

lettucemode commented 10 years ago

Added a notification for promo cards and added "built by the OCTGN SBB" as shown above.

sIKE23 commented 10 years ago

Couple of items, 1 a typo, You muse successfully validate a deck before exporting to a forum post.

Muse = Must

Best Deck NA = OCTGN Deck (I like to advertise)

Not all Promos have a CardID with Promo the first three were just MWPRO so if (Property(card, "Set").Contains("Promo")) will not work all of the time, I noticed as I was using the Dispel Promo for testing, which happens to be MWPRO1.

lettucemode commented 10 years ago

The "Set" field contains which expansion the card came in, for example "Druid vs. Necromancer" or "Core". It is not the CardID field. Right now all the promo cards have a Set of "Promo" so I think the code is fine as-is.

lettucemode commented 10 years ago

Made the requested updates.

sIKE23 commented 10 years ago

The first two items are fixed, but it didn't warn me on the Promo in the Spellbook.

One other minor nit since you have to recompile:

text.AppendLine("[mage]A " + card.Name + " book[/mage]");

would you change book to Spellbook, I have added in a Promo Card Spellbook for testing this issue.

lettucemode commented 10 years ago

Done and done.

sIKE23 commented 10 years ago

lettucemoe,

Everything looks great, have a fun weekend!

On Wed, Feb 12, 2014 at 8:57 PM, Steven Hyland notifications@github.comwrote:

Done and done.

— Reply to this email directly or view it on GitHubhttps://github.com/sIKE23/Mage-Wars/issues/4#issuecomment-34943827 .

lettucemode commented 10 years ago

Thanks sIKE, you too. Keep them forum-ites in line for me.

On Thu, Feb 13, 2014 at 1:54 AM, sIKE23 notifications@github.com wrote:

lettucemoe,

Everything looks great, have a fun weekend!

On Wed, Feb 12, 2014 at 8:57 PM, Steven Hyland notifications@github.comwrote:

Done and done.

Reply to this email directly or view it on GitHub< https://github.com/sIKE23/Mage-Wars/issues/4#issuecomment-34943827> .

Reply to this email directly or view it on GitHubhttps://github.com/sIKE23/Mage-Wars/issues/4#issuecomment-34952965 .

May your forehead grow like the mighty oak.