Skip to content

fix: Улучшена обработка ошибок в RoleConverter и ExtendXStream #568

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.

Sign up for GitHub

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

Merged
theshadowco merged 3 commits into from
Jan 8, 2026
Merged

fix: Улучшена обработка ошибок в RoleConverter и ExtendXStream #568

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
d766b5c
fix: Улучшена обработка ошибок в RoleConverter и ExtendXStream
johnnyshut Jan 7, 2026
4210601
refactor: Обновление логирования ошибок в ExtendXStream с использован…
johnnyshut Jan 8, 2026
96a415f
feat: Добавлены комментарии и документация в RoleConverter для улучше…
johnnyshut Jan 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions src/main/java/com/github/_1c_syntax/bsl/reader/common/xstream/ExtendXStream.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -123,11 +123,13 @@ public Object fromXML(File file) {
try {
result = super.fromXML(file);
} catch (ConversionException e) {
LOGGER.error("Can't read file '{}' - it's broken (skipped) \n", file, e);
LOGGER.warn("Can't read file '{}' - it's broken (skipped) \n", file, e);
} catch (CannotResolveClassException e) {
LOGGER.debug("Can't read file '{}' - unknown class (skipped) \n", file, e);
} catch (StreamException e) {
LOGGER.error("Can't read file '{}' - it's broken (skipped): {}", file, e.getCause().getMessage());
LOGGER.warn("Can't read file '{}' - it's broken (skipped)", file, e);
} catch (Exception e) {
LOGGER.warn("Can't read file '{}' - unexpected error (skipped): {}", file, e.getMessage(), e);
}
}
return result;
Expand Down
31 changes: 30 additions & 1 deletion src/main/java/com/github/_1c_syntax/bsl/reader/designer/converter/RoleConverter.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,17 +31,33 @@
import java.nio.file.Path;
import java.nio.file.Paths;

/**
* Конвертер для чтения ролей из формата конфигуратора
*/
@DesignerConverter
public class RoleConverter extends AbstractReadConverter {

private static final String DATA_FIELD = "data";

/**
* Выполняет чтение роли из XML, включая данные прав доступа из файла Rights.xml
*
* @param reader Ридер XML потока
* @param context Контекст десериализации
* @return Прочитанный объект роли с данными прав доступа
*/
@Override
public Object unmarshal(HierarchicalStreamReader reader, UnmarshallingContext context) {
var readerContext = super.read(reader, context);
RoleData data;
try {
data = (RoleData) ExtendXStream.read(reader, dataPath(readerContext.getCurrentPath(), readerContext.getName()));
var readResult = ExtendXStream.read(reader, dataPath(readerContext.getCurrentPath(), readerContext.getName()));
if (readResult instanceof RoleData roleData) {
data = roleData;
} else {
// файл не прочитан или прочитан некорректно
data = RoleData.EMPTY;
}
} catch (Exception e) {
// ничего не делаем, считаем файл битым
data = RoleData.EMPTY;
Expand All @@ -51,11 +67,24 @@ public Object unmarshal(HierarchicalStreamReader reader, UnmarshallingContext co
return readerContext.build();
}

/**
* Проверяет, может ли конвертер обработать указанный тип
*
* @param type Тип класса для проверки
* @return true, если тип является Role или его подклассом
*/
@Override
public boolean canConvert(Class type) {
return Role.class.isAssignableFrom(type);
}

/**
* Формирует путь к файлу Rights.xml для роли
*
* @param path Путь к файлу описания роли
* @param name Имя роли
* @return Путь к файлу Rights.xml
*/
private static Path dataPath(Path path, String name) {
return Paths.get(path.getParent().toString(), name, "Ext", "Rights.xml");
}
Expand Down
30 changes: 29 additions & 1 deletion src/main/java/com/github/_1c_syntax/bsl/reader/edt/converter/RoleConverter.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,17 +30,33 @@

import java.nio.file.Path;

/**
* Конвертер для чтения ролей из формата ЕДТ
*/
@EDTConverter
public class RoleConverter extends AbstractReadConverter {

private static final String DATA_FIELD = "data";

/**
* Выполняет чтение роли из XML, включая данные прав доступа из файла Rights.rights
*
* @param reader Ридер XML потока
* @param context Контекст десериализации
* @return Прочитанный объект роли с данными прав доступа
*/
@Override
public Object unmarshal(HierarchicalStreamReader reader, UnmarshallingContext context) {
var readerContext = super.read(reader, context);
RoleData data;
try {
data = (RoleData) ExtendXStream.read(reader, dataPath(readerContext.getCurrentPath()));
var readResult = ExtendXStream.read(reader, dataPath(readerContext.getCurrentPath()));
if (readResult instanceof RoleData roleData) {
data = roleData;
} else {
// файл не прочитан или прочитан некорректно
data = RoleData.EMPTY;
}
} catch (Exception e) {
// ничего не делаем, считаем файл битым
data = RoleData.EMPTY;
Expand All @@ -50,11 +66,23 @@ public Object unmarshal(HierarchicalStreamReader reader, UnmarshallingContext co
return readerContext.build();
}

/**
* Проверяет, может ли конвертер обработать указанный тип
*
* @param type Тип класса для проверки
* @return true, если тип является Role или его подклассом
*/
@Override
public boolean canConvert(Class type) {
return Role.class.isAssignableFrom(type);
}

/**
* Формирует путь к файлу Rights.rights для роли
*
* @param path Путь к файлу описания роли
* @return Путь к файлу Rights.rights
*/
private static Path dataPath(Path path) {
return Path.of(path.getParent().toString(), "Rights.rights");
}
Expand Down
Loading