Skip to content

Commit e45a242

Browse files
committed
Implement some PR review suggestions
1 parent fc78f17 commit e45a242

File tree

4 files changed

+4
-9
lines changed

4 files changed

+4
-9
lines changed

src/euphorie/client/navigation.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ def FindNextQuestion(after, dbsession, filter=None):
4141
model.Choice.id == model.SurveyTreeItem.id,
4242
sql.or_(
4343
model.Choice.condition == None, # noqa: E711
44-
model.Choice.condition.like(model.Option.zodb_path),
44+
model.Choice.condition == model.Option.zodb_path, # noqa: E711
4545
),
4646
),
4747
),

src/euphorie/content/browser/choice.py

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,6 @@
66

77

88
class ChoiceView(DefaultView, DragDropHelper):
9-
@property
10-
@memoize
11-
def my_context(self):
12-
return aq_inner(self.context)
13-
149
@property
1510
def options(self):
1611
return [
@@ -19,6 +14,6 @@ def options(self):
1914
"url": option.absolute_url(),
2015
"title": option.title,
2116
}
22-
for option in self.my_context.values()
17+
for option in self.context.values()
2318
if IOption.providedBy(option)
2419
]

src/euphorie/upgrade/content/v1/20250402124248_add_content_type__choice/types.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
<object meta_type="Conditional Dexterity FTI"
44
name="euphorie.choice"
55
/>
6-
<object meta_type="Conditional Dexterity FTI"
6+
<object meta_type="Dexterity FTI"
77
name="euphorie.option"
88
/>
99
<object meta_type="Conditional Dexterity FTI"

src/euphorie/upgrade/content/v1/20250402124248_add_content_type__choice/types/euphorie.choice.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
<?xml version="1.0" encoding="utf-8"?>
22
<object xmlns:i18n="http://xml.zope.org/namespaces/i18n"
3-
meta_type="Dexterity FTI"
3+
meta_type="Conditional Dexterity FTI"
44
name="euphorie.choice"
55
i18n:domain="euphorie"
66
>

0 commit comments

Comments
 (0)