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

Checks to ReportViewer for export in file if that file already exists #206

Merged
merged 3 commits into from
Oct 20, 2021

Conversation

DobriyMisha
Copy link
Contributor

Добавлена кастомная проверка на существование файла с таким именем в директории при сохранении выгрузки
DoOverwrite не работала в данном случае, так как невозможно было присвоить имени файла необходимое расширение ДО выполнения процедуры проверки DoOverwriteConfirmation, следовательно процедура не замечает существования дубликата файла так как текущий сохраняемый файл не имеет расширения в своем имени
Также по задаче необходимо было изменить локализацию кнопок и заголовка, сделано это через ресурсы RdlEngine (чтобы не иметь кириллицы в коде проекта).
Для доступа к ресурсам из другого проекта пришлось модификаторы доступа ресурсов сделать public.

DobriyMisha and others added 3 commits September 24, 2021 16:25
…e name)

Different localization for export dialog and patterns for filter by file type

# Conflicts was:
#	RdlGtkViewer/RdlGtkViewer/ReportViewer.cs
Resolve conflict, duplicated </Project>
fc.AddFilter(csvFilter);
}

if(!restrictedOutputPresentationTypes.Contains(OutputPresentationType.ExcelTableOnly)) {
Gtk.FileFilter excel2007Data = new Gtk.FileFilter { Name = "Excel no formatting (Fast)" };
Gtk.FileFilter excel2007Data = new Gtk.FileFilter { Name = "Excel без форматирования (Быстро)" };
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why this translation don't use resourses? Default language is English.

@Gankov Gankov merged commit 832080a into majorsilence:master Oct 20, 2021
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.

2 participants