Skip to content
This repository has been archived by the owner on Oct 29, 2022. It is now read-only.

Add Plot buy/selling #343

Merged
merged 31 commits into from
Oct 6, 2020
Merged

Add Plot buy/selling #343

merged 31 commits into from
Oct 6, 2020

Conversation

DerDevHD
Copy link
Contributor

@DerDevHD DerDevHD commented Oct 5, 2020

Introduction

Hi. Y'all waited for Plot selling and buying. And now its here!

Relevant issues

#9

Changes

API changes

There are added 2 new Methods for the MyPlot class:

MyPlot::sellPlot(Plot $plot, float $price)
MyPlot::buyPlot(Plot $plot, Player $player)

Behavioral changes

Economy is now registered onLoad instead of onEnable

Backwards compatibility

It's 100% backwards compatible

Follow-up

  • Translations Needed
  • Add buy and sell forms

Tests

Tested it successfully without any errors.

@jasonw4331 jasonw4331 added PR: Feature Additions PR: Translation This Pull Request adds or improves a language translation labels Oct 5, 2020
@jasonw4331 jasonw4331 assigned jasonw4331 and DerDevHD and unassigned jasonw4331 Oct 5, 2020
@jasonw4331 jasonw4331 linked an issue Oct 5, 2020 that may be closed by this pull request
27 tasks
resources/eng.ini Outdated Show resolved Hide resolved
@jasonw4331 jasonw4331 merged commit 70e6840 into jasonw4331:master Oct 6, 2020
@DerDevHD DerDevHD deleted the todo/economy branch October 9, 2020 10:57
@HimbeersaftLP
Copy link
Contributor

Doesn't this allow you to buy more plots than the maximum plot limit per player?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR: Feature Additions PR: Translation This Pull Request adds or improves a language translation Priority: Low Status: Work In Progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Never Ending TODO List
3 participants