Skip to content

Commit c698e1d

Browse files
committed
Fix logic for unpinning articles
1 parent 43e0e4e commit c698e1d

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

pgweb/news/management/commands/social_post.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,8 @@ def handle(self, *args, **options):
5454
# Pin or unpin any articles as needed
5555
pna = PinnedNewsArticle.objects.select_related('pinnedarticle').only('pinnedarticle', 'pinnedtoproviders', 'pinnedarticle__postedto').all()[0]
5656
for p in allproviders:
57-
if pna.pinnedtoproviders.get(p.name, None) != pna.pinnedarticle.postedto.get(p.name, None):
58-
if p.set_pin(pna.pinnedarticle.postedto.get(p.name, None)):
59-
pna.pinnedtoproviders[p.name] = pna.pinnedarticle.postedto.get(p.name, None)
57+
pinnedid = pna.pinnedarticle.postedto.get(p.name, None) if pna.pinnedarticle else None
58+
if pna.pinnedtoproviders.get(p.name, None) != pinnedid:
59+
if p.set_pin(pinnedid):
60+
pna.pinnedtoproviders[p.name] = pinnedid
6061
pna.save(update_fields=['pinnedtoproviders'])

0 commit comments

Comments
 (0)