#92 display_attr() should return str not bytes in py3
Closed: Fixed 6 years ago Opened 6 years ago by ilias95.

In python3 display_attr() method of mapped object returns bytes. We should make it return str instead.


Metadata Update from @ilias95:
- Issue assigned to ilias95

6 years ago

Metadata Update from @ilias95:
- Assignee reset

6 years ago

Metadata Update from @ilias95:
- Issue assigned to ilias95

6 years ago

Metadata Update from @ilias95:
- Custom field Origin adjusted to None
- Custom field Review Status adjusted to None

6 years ago

Metadata Update from @ilias95:
- Custom field Review Status adjusted to review (was: None)

6 years ago

There is an extra bug actually. get_attr_vals_type() functions call a non-existing method due to a typo.

They call get_attrs_val() but this method does not exist.
Instead the following two methods exist:

  1. get_attr_vals -> https://pagure.io/lib389/blob/master/f/lib389/_mapped_object.py#_322
  2. get_attrs_vals -> https://pagure.io/lib389/blob/master/f/lib389/_mapped_object.py#_314

I'm not sure what is the difference between these 2 and I chose to go with the first one, but please make sure it is the one we want.

get_attr_vals I think was get 1 attribute and many values, and get_attrs_vals was get many attributes with many values (I think).

Okay, so the patch should be fine I think then.

commit d304445
To ssh://git@pagure.io/lib389.git
2a50b9d..709d572 master -> master

Metadata Update from @ilias95:
- Issue close_status updated to: Fixed
- Issue status updated to: Closed (was: Open)

6 years ago

Login to comment on this ticket.

Metadata