MTSR/tech_additions
4
0
Fork 2
Code Issues 4 Pull Requests Actions Packages Projects Releases Wiki Activity

fix: fix dupe airblock #5

Merged
Koldun merged 4 commits from air-fix into devel 2024-12-22 15:31:17 +03:00
Conversation 13 Commits 4 Files Changed 1 +19 -5
shepel commented 2024-12-22 11:31:45 +03:00
Owner
Copy Link

Добавил проверку на права креатива у игрока при установке air блока.
Эта правка исключает дюп блоков воздуха.

Добавил проверку на права креатива у игрока при установке air блока. Эта правка исключает дюп блоков воздуха.
shepel added 1 commit 2024-12-22 11:31:46 +03:00
shepel requested review from Reload 2024-12-22 11:31:57 +03:00
shepel requested review from Koldun 2024-12-22 11:31:57 +03:00
Koldun commented 2024-12-22 11:52:45 +03:00
Owner
Copy Link

Мне кажется, надо просто добавить вызов метода itemstack.take_item, чтобы эти блоки тратились.

А позицию проверять с помощью minetest.is_protected.

Тогда блок можно будет использовать обычным игрокам (без creative), но зато без дюпа

Мне кажется, надо просто добавить вызов метода itemstack.take_item, чтобы эти блоки тратились. А позицию проверять с помощью minetest.is_protected. Тогда блок можно будет использовать обычным игрокам (без creative), но зато без дюпа
Koldun added the
fix
 label 2024-12-22 11:53:17 +03:00
shepel commented 2024-12-22 11:57:23 +03:00
Author
Owner
Copy Link

@Koldun , только это не решит проблему установки в воздухе (без прикрепления к соседнему блоку), а также проблему установки на заприваченых территориях.
А после моей правки игрок сможет ставить блок, но только там где можно и с примыканием к соседнему блоку.
Протестируй на локалке, плиз, работает норм.

@Koldun , только это не решит проблему установки в воздухе (без прикрепления к соседнему блоку), а также проблему установки на заприваченых территориях. А после моей правки игрок сможет ставить блок, но только там где можно и с примыканием к соседнему блоку. Протестируй на локалке, плиз, работает норм.
Koldun commented 2024-12-22 13:00:02 +03:00
Owner
Copy Link

только это не решит проблему установки в воздухе (без прикрепления к соседнему блоку)

Как я понял, это не баг, а фича

> только это не решит проблему установки в воздухе (без прикрепления к соседнему блоку) Как я понял, это не баг, а фича
Koldun commented 2024-12-22 13:00:39 +03:00
Owner
Copy Link

а также проблему установки на заприваченых территориях.

Это решается с помощью упомянтой функции minetest.is_protected

> а также проблему установки на заприваченых территориях. Это решается с помощью упомянтой функции `minetest.is_protected`
Koldun commented 2024-12-22 13:01:33 +03:00
Owner
Copy Link

А после моей правки игрок сможет ставить блок

Только игрок с креативом

> А после моей правки игрок сможет ставить блок Только игрок с креативом
shepel commented 2024-12-22 13:03:04 +03:00
Author
Owner
Copy Link

только это не решит проблему установки в воздухе (без прикрепления к соседнему блоку)

Как я понял, это не баг, а фича

Которая не должна быть доступна игроку без креатива.

> > только это не решит проблему установки в воздухе (без прикрепления к соседнему блоку) > > Как я понял, это не баг, а фича > > Которая не должна быть доступна игроку без креатива.
shepel commented 2024-12-22 13:03:31 +03:00
Author
Owner
Copy Link

А после моей правки игрок сможет ставить блок

Только игрок с креативом

Нет. Проверь.

> > А после моей правки игрок сможет ставить блок > > Только игрок с креативом > Нет. Проверь.
Koldun commented 2024-12-22 13:07:42 +03:00
Owner
Copy Link

А после моей правки игрок сможет ставить блок

Только игрок с креативом

Нет. Проверь.

А ну да, когда блок ставится не в пустоту, там другая функция вызывается. Тогда этот вопрос снимается

> > > А после моей правки игрок сможет ставить блок > > > > Только игрок с креативом > > > > Нет. Проверь. А ну да, когда блок ставится не в пустоту, там другая функция вызывается. Тогда этот вопрос снимается
shepel commented 2024-12-22 13:12:26 +03:00
Author
Owner
Copy Link

А после моей правки игрок сможет ставить блок

Только игрок с креативом

Нет. Проверь.

А ну да, когда блок ставится не в пустоту, там другая функция вызывается. Тогда этот вопрос снимается

Это как раз не даст игроку ставить блок в приватных местах.

> > > > А после моей правки игрок сможет ставить блок > > > > > > Только игрок с креативом > > > > > > > Нет. Проверь. > > А ну да, когда блок ставится не в пустоту, там другая функция вызывается. Тогда этот вопрос снимается Это как раз не даст игроку ставить блок в приватных местах.
Koldun commented 2024-12-22 13:19:46 +03:00
Owner
Copy Link

Потестировал. Функцию minetest.is_protected всё равно надо добавить. Иначе игрок с креативом имеет возможность установить блок в пустоту в чужом привате

Потестировал. Функцию `minetest.is_protected` всё равно надо добавить. Иначе игрок с креативом имеет возможность установить блок в пустоту в чужом привате
shepel added 1 commit 2024-12-22 14:07:59 +03:00
shepel commented 2024-12-22 14:13:29 +03:00
Author
Owner
Copy Link

Потестировал. Функцию minetest.is_protected всё равно надо добавить. Иначе игрок с креативом имеет возможность установить блок в пустоту в чужом привате

Готовченко.

Протестировал:
Игрок ставит только с примыканием, блок тратится. Не поставить в защищенном месте.
Игрок с креативом ставит с примыканием и в воздухе, блок не тратится. Не поставить в защищенном месте.
Админ ставит как угодно.

> Потестировал. Функцию `minetest.is_protected` всё равно надо добавить. Иначе игрок с креативом имеет возможность установить блок в пустоту в чужом привате Готовченко. Протестировал: Игрок ставит только с примыканием, блок тратится. Не поставить в защищенном месте. Игрок с креативом ставит с примыканием и в воздухе, блок не тратится. Не поставить в защищенном месте. Админ ставит как угодно.
Koldun added 2 commits 2024-12-22 15:26:11 +03:00
Koldun changed target branch from lsr to devel 2024-12-22 15:28:59 +03:00
Koldun force-pushed air-fix from 8064991b53 to 0010d400e0 2024-12-22 15:29:35 +03:00 Compare
Koldun approved these changes 2024-12-22 15:30:16 +03:00
Koldun merged commit 168180ea34 into devel 2024-12-22 15:31:17 +03:00
Koldun deleted branch air-fix 2024-12-22 15:31:17 +03:00
Koldun referenced this issue from a commit 2024-12-22 15:31:18 +03:00
fix: fix dupe airblock (#5)
Koldun commented 2024-12-22 15:32:42 +03:00
Owner
Copy Link

@Reload у нас тут обновление

@Reload у нас тут обновление
Reload was assigned by Koldun 2024-12-22 15:33:05 +03:00
Reload commented 2024-12-23 09:57:57 +03:00
Owner
Copy Link

@Koldun переполз на devel ветку

@Koldun переполз на devel ветку
👍 1
Sign in to join this conversation.
Reviewers
No reviewers
Reload
Koldun
Labels
Clear labels
  
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
fix

fix a bug

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

No Label
bug
duplicate
enhancement
fix
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
Reload
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: MTSR/tech_additions#5
No description provided.
Delete Branch "air-fix"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?