-
Notifications
You must be signed in to change notification settings - Fork 21
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
Добавлена поддержка прав доступа хранящихся в ролях #125
Conversation
import com.github._1c_syntax.mdclasses.mdo.WSReference; | ||
import com.github._1c_syntax.mdclasses.mdo.WebService; | ||
import com.github._1c_syntax.mdclasses.mdo.XDTOPackage; | ||
import com.github._1c_syntax.mdclasses.mdo.*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Без импорта через *
import com.github._1c_syntax.mdclasses.mdo.Template; | ||
import com.github._1c_syntax.mdclasses.mdo.WEBServiceOperation; | ||
import com.github._1c_syntax.mdclasses.mdo.WebService; | ||
import com.github._1c_syntax.mdclasses.mdo.*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Без импорта через *
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@otymko готово
import com.github._1c_syntax.mdclasses.metadata.additional.MDOType; | ||
import com.github._1c_syntax.mdclasses.metadata.additional.ModuleType; | ||
import com.github._1c_syntax.mdclasses.metadata.additional.ScriptVariant; | ||
import com.github._1c_syntax.mdclasses.metadata.additional.*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Без импорта через *
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@otymko готово
import java.util.List; | ||
import java.util.Map; | ||
import java.util.Optional; | ||
import java.util.*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
без импорта через *
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@otymko готово
@EightM давай заведем ишузу на "виды прав". Имена можно вытянуть из xsd схем. |
3a3683e
to
bf12b52
Compare
@Setter | ||
public class RoleData { | ||
|
||
@XStreamAlias("setForNewObjects") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
давай откомментируем эти свойства
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@otymko готово
@@ -250,6 +251,7 @@ private void addClassAliases(XStream xStream) { | |||
xStream.alias("mdclass:Enum", MDOEnum.class); | |||
xStream.alias("mdclass:Report", Report.class); | |||
xStream.alias("mdclass:Role", Role.class); | |||
xStream.alias("Rights", RoleData.class); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
раз уж в xml поле зовется rights, может и RoleData назвать так же?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
именование сделано по формату: form-> formData, role->roleData
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
для роли наверное стоит засунуть права в roleData
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Так и сделано
… feature/RoleData � Conflicts: � src/main/java/com/github/_1c_syntax/mdclasses/utils/MDOFactory.java
Частично #108
Добавлены данные по правам роли: