magefree / magefree/mage

Replace card.putOntoBattlefield by player.moveCard...

Open
#4,866 6 comments 0 reactions 1 assignee Claimed by @LevelX2 View on GitHub
refactoring
Dominant language
Java
Stars
2.4k
Forks
940
Avg merge
2d 12h
Merged PRs (30d)
160

Description

There are still a lot of cards or framwork classes that use not the correct methods to move cards from one zone to another.

If possible always use the player.moveCards... methods, that move 1 or more cards at once and also create the correct triggers for movement of more than one card at the same time.
And to move to library the player.putOn[Top/Button]OfLibrary... methods.

- [x] Remove card.putOntoBattlefield usages if possible by player.moveCards methods.
- [x] moveToZone(Zone.LIBRARY (25x)
- [x] moveToZone(Zone.EXILED (3x)
- [x] moveToZone(Zone.HAND (71x)
- [x] moveToZone(Zone.BATTLEFIELD (11x)
- [x] moveToZone(Zone.GRAVEYARD (2x)
- [x] moveToZone({other}(9x)

- [ ] moveCardToExileWithInfo(... (142x)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.