Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SetAttr: fix recursive file owner changing + wrong group mark indicating + recursive file attributes/flags changing #2656

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

spnethw
Copy link
Contributor

@spnethw spnethw commented Feb 3, 2025

Диалог Ctrl-A:

  1. Неправильно устанавливался владелец при рекурсивной обработке вложенных папок (вместо имени владельца использовалось имя группы).
  2. Некорректная индикация звёздочкой при изменении группы (вместо сравнения с именем исходной группы происходило сравнение с именем исходного владельца).
  3. Рекурсивное выставление атрибутов (Immutable / Append / Hidden) для вложенных папок должно было происходить, но не отрабатывало потому, что было размещено не под той веткой if, и выполнялось только в том случае, если одновременно менялись права доступа.

@spnethw spnethw changed the title SetAttr: fix recursive file owner changing + wrong group mark indicating SetAttr: fix recursive file owner changing + wrong group mark indicating + recursive file attributes/flags changing Feb 9, 2025
unxed added a commit to unxed/far2l that referenced this pull request Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant