IntellectualSites / FastAsyncWorldEdit

Blazingly fast world manipulation for artists, builders and everyone else: https://www.spigotmc.org/resources/13932/
Other
603 stars 214 forks source link

Connections Command #313

Open CJCrafter opened 4 years ago

CJCrafter commented 4 years ago

Issue

Whenever you paste in a block with connections (Example: //paste spruce_fence, //replace air iron_bars), the block does not have any connections.

Solution

I would like a //update command. This would update every single block in a region and set connections if it should have connections.

Alternate explanation:

foreach (x in region)
    foreach (y in region)
        foreach (z in region)
            if (block can have connections) update connections
        rof
    rof
rof

Need a 3rd one? From this:

to this:

Extra

Possible arguments are //update dir[south,west,north,east] which could specify which directions are allowed.

Blocks that should have connections:

CJCrafter commented 4 years ago

Paintsplatters suggested to add:

//paste -u

(Or maybe -update)


This would essentially run //paste then //update, effectively automatically updating every block inside the Player's clipboard, and pasting.

stale[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

RedstoneFuture commented 3 years ago

Basically I like this idea. No matter all praxis problems at here. Maybe the normal direction check are simple to implement.

This would essentially run //paste then //update, effectively automatically updating every block inside the Player's clipboard, and pasting.

Bukkit has more than one update behaviors, as I know. So it needs tests to find the right way to update the blocks correctly.

Sentropic commented 2 years ago

I think this is a huge need. I often enjoy the feature of avoiding block updates (like using Lapisdemon's Bonus), but still run into situations where this command would be really useful.

VL4DST3R commented 2 months ago

I see a pull request for this has been made for a while now, when will it be merged? This is one of the most important features this amazing plugin has been lacking for years now!

dordsor21 commented 2 months ago

I see a pull request for this has been made for a while now, when will it be merged? This is one of the most important features this amazing plugin has been lacking for years now!

Once it's been reviewed + approved ¯\_(ツ)_/¯