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.

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

Jump to bottom

Fixed the displaying of optional attributes #432

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
+33 −9
Open

Fixed the displaying of optional attributes #432

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
06ae16c
Fixed the displaying of optional attributes
DimDone Jul 16, 2024
7cc8ffd
Fixed the displaying optional attributes after reset action
DimDone Jul 24, 2024
eff7cb2
Added copy action to list of attributes
DimDone Jul 26, 2024
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
2 changes: 1 addition & 1 deletion src/adldap/ad_config.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -558,7 +558,6 @@ AttributeType AdConfig::get_attribute_type(const QString &attribute) const {

const QString attribute_syntax = schema.get_string(ATTRIBUTE_ATTRIBUTE_SYNTAX);
const QString om_syntax = schema.get_string(ATTRIBUTE_OM_SYNTAX);

Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove that empty change please

if (type_map.contains(attribute_syntax) && type_map[attribute_syntax].contains(om_syntax)) {
return type_map[attribute_syntax][om_syntax];
} else {
Expand All @@ -576,6 +575,7 @@ LargeIntegerSubtype AdConfig::get_attribute_large_integer_subtype(const QString
ATTRIBUTE_LOCKOUT_TIME,
ATTRIBUTE_BAD_PWD_TIME,
ATTRIBUTE_CREATION_TIME,
ATTRIBUTE_MSDS_USER_PASSWORD_EXPIRY_TIME_COMPUTED,
};
static const QList<QString> timespans = {
ATTRIBUTE_MAX_PWD_AGE,
Expand Down
2 changes: 2 additions & 0 deletions src/adldap/ad_defines.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -205,6 +205,8 @@ enum SystemFlagsBit {
#define ATTRIBUTE_OBJECT_VERSION "objectVersion"
#define ATTRIBUTE_SERVER_REFERENCE "serverReference"
#define ATTRIBUTE_SERVER_REFERENCE_BL "serverReferenceBL"
#define ATTRIBUTE_MSDS_USER_PASSWORD_EXPIRY_TIME_COMPUTED "msDS-UserPasswordExpiryTimeComputed"
#define ATTRIBUTE_MSDS_USER_ACCOUNT_CONTROL_COMPUTED "msDS-User-Account-Control-Computed"


#define CLASS_GROUP "group"
Expand Down
4 changes: 2 additions & 2 deletions src/adldap/ad_display.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@
#include <QList>
#include <QString>
#include <algorithm>
#include <QTextStream>
Copy link
Collaborator

Choose a reason for hiding this comment

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

why include it if you don't use it?


const qint64 SECONDS_TO_MILLIS = 1000LL;
const qint64 MINUTES_TO_SECONDS = 60LL;
Expand Down Expand Up @@ -63,7 +64,7 @@ QString attribute_display_value(const QString &attribute, const QByteArray &valu
return samaccounttype_to_display_value(value);
} else if (attribute == ATTRIBUTE_PRIMARY_GROUP_ID) {
return primarygrouptype_to_display_value(value);
} else if (attribute == ATTRIBUTE_GROUP_TYPE || attribute == ATTRIBUTE_SYSTEM_FLAGS) {
} else if (attribute == ATTRIBUTE_GROUP_TYPE || attribute == ATTRIBUTE_SYSTEM_FLAGS || attribute == ATTRIBUTE_MSDS_USER_ACCOUNT_CONTROL_COMPUTED) {
return attribute_hex_displayed_value(attribute, value);
} else if (attribute == ATTRIBUTE_MS_DS_SUPPORTED_ETYPES) {
return msds_supported_etypes_to_display_value(value);
Expand All @@ -73,7 +74,6 @@ QString attribute_display_value(const QString &attribute, const QByteArray &valu
}
case AttributeType_LargeInteger: {
const LargeIntegerSubtype subtype = adconfig->get_attribute_large_integer_subtype(attribute);

switch (subtype) {
case LargeIntegerSubtype_Datetime: return large_integer_datetime_display_value(attribute, value, adconfig);
case LargeIntegerSubtype_Timespan: return timespan_display_value(value);
Expand Down