Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1326)

Issue 13240047: User subscription settings with item name appended to the itemid subscriptions (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 8 months ago by ana.balica
Modified:
10 years, 3 months ago
Reviewers:
thomas.j.waldmann
Visibility:
Public.

Description

Now the subscriptions looks like this: itemid:7bcf8764a2ce4e7e80f18458fcea1b18 (Home) If the item will be renamed, than the new name will appear. If item was deleted or completely destroyed, then it will display itemid:7bcf8764a2ce4e7e80f18458fcea1b18 (This item doesn't exist)

Patch Set 1 #

Total comments: 5

Patch Set 2 : Better docstring #

Patch Set 3 : Retrieve item name easier and change name0 to name_ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+38 lines, -2 lines) Patch
M MoinMoin/forms.py View 1 2 1 chunk +38 lines, -2 lines 0 comments Download

Messages

Total messages: 2
Thomas.J.Waldmann
https://codereview.appspot.com/13240047/diff/1/MoinMoin/forms.py File MoinMoin/forms.py (right): https://codereview.appspot.com/13240047/diff/1/MoinMoin/forms.py#newcode161 MoinMoin/forms.py:161: appends to the end of ITEMID subscriptions the name ...
10 years, 8 months ago (2013-09-09 11:37:56 UTC) #1
ana.balica
10 years, 8 months ago (2013-09-10 21:53:00 UTC) #2
https://codereview.appspot.com/13240047/diff/1/MoinMoin/forms.py
File MoinMoin/forms.py (right):

https://codereview.appspot.com/13240047/diff/1/MoinMoin/forms.py#newcode190
MoinMoin/forms.py:190: return self.separator.join(subscriptions)
On 2013/09/09 11:37:56, Thomas.J.Waldmann wrote:
> in general, i am not sure whether this patching around is the best way to
solve
> this problem. maybe ask jek (from flatland project).

So there are some other ways to solve the problem, but they lead to some extra
complex stuff too. For instance, we can use
List.of(String.using(default_factory=better_subscriptions) where
better_subscriptions is a function that will append the names (same thing as u()
does). But we need to strip this afterwards, in validator for example (here we
do it in value()). Also we need to rewrite the printing (a List outputs a raw
list), but for that we can easily use MyJoinedString instead of List.

Generally it might be a good practice to use a compound and subclass it if it
fits our needs. 

Moreover we will probably change this settings appearance after the UI changes.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b