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

Refactoring posting_attachments #112

Merged
merged 1 commit into from Jan 22, 2017

Conversation

Projects
None yet
3 participants
@VasyOk
Contributor

VasyOk commented Jan 16, 2017

Небольшой рефакторинг.
Исправление https://torrentpier.me/forum/threads/sozdanie-i-redaktirovanie-temy.41621/

@VasyOk VasyOk added the Enhancement label Jan 16, 2017

@VasyOk VasyOk added this to the Версия 2.1.6 milestone Jan 16, 2017

@VasyOk VasyOk self-assigned this Jan 16, 2017

@VasyOk VasyOk requested review from Exileum and leroy0 Jan 16, 2017

bb_die('Could not create filename for attachment');
}
$this->attach_filename = $new_physical_filename;

This comment has been minimized.

@Exileum

Exileum Jan 16, 2017

Member

Мне совершенно отказывает память, но я не помню по какой причине оно стояло там, где стоит. Надо проверить редактирование и перезаливку торрент-файла, а точнее что с именем файла все нормально остается.

@Exileum

Exileum Jan 16, 2017

Member

Мне совершенно отказывает память, но я не помню по какой причине оно стояло там, где стоит. Надо проверить редактирование и перезаливку торрент-файла, а точнее что с именем файла все нормально остается.

@Exileum

Спасибо, в целом все хорошо, но нужно вернуть операнды на место, в результате обсуждения в Slack пришли к выводу, что такой порядок совершенно излишен и создает только изменения взятые за одно. Ну и пара мелких косяков.

@Exileum Exileum added the Bug label Jan 22, 2017

@Exileum Exileum merged commit a093915 into torrentpier:master Jan 22, 2017

1 of 2 checks passed

SensioLabsInsight Code quality below expectations.
Details
continuous-integration/styleci/pr The StyleCI analysis has passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment