[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-09-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #26 from Mark Holmquist mtrac...@member.fsf.org ---
I believe the two patches fix this issue. If there are further caching issues
with the campaign contents, let us know and we can reopen. :)

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-09-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

Mark Holmquist mtrac...@member.fsf.org changed:

   What|Removed |Added

 Status|PATCH_TO_REVIEW |RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-08-27 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #22 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 156692 had a related patch set uploaded by MarkTraceur:
Add start/end and whileActive to the campaigns

https://gerrit.wikimedia.org/r/156692

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-08-27 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #23 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 156692 merged by jenkins-bot:
Add start/end and whileActive to the campaigns

https://gerrit.wikimedia.org/r/156692

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-08-27 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #24 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 156694 had a related patch set uploaded by MarkTraceur:
Add beforeActive and afterActive to campaigns

https://gerrit.wikimedia.org/r/156694

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-08-27 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #25 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 156694 merged by jenkins-bot:
Add beforeActive and afterActive to campaigns

https://gerrit.wikimedia.org/r/156694

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-08-26 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #21 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 155351 merged by jenkins-bot:
Add beforeActive and afterActive to campaigns

https://gerrit.wikimedia.org/r/155351

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-08-26 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #20 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 153818 merged by jenkins-bot:
Add start/end and whileActive to the campaigns

https://gerrit.wikimedia.org/r/153818

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-08-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #19 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 155351 had a related patch set uploaded by MarkTraceur:
Add beforeActive and afterActive to campaigns

https://gerrit.wikimedia.org/r/155351

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-08-13 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #17 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 153818 had a related patch set uploaded by MarkTraceur:
Add start/end and whileActive to the campaigns

https://gerrit.wikimedia.org/r/153818

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-08-13 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

Gerrit Notification Bot gerritad...@wikimedia.org changed:

   What|Removed |Added

 Status|NEW |PATCH_TO_REVIEW

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-08-13 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #18 from Mark Holmquist mtrac...@member.fsf.org ---
I think this patch will just about do the trick.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-06-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #15 from Romaine romaine.w...@gmail.com ---
I hope the Multimedia team is interested in the ideas of the UploadWizard from
the users who use them?

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-06-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #16 from Romaine romaine.w...@gmail.com ---
(In reply to Gilles Dubuc from comment #11)
 Yes, that sounds like a much better idea. Basically define the start and end
 date in the campaign settings, and have optional content/template settings
 to display before, during and after. I'll file it as a task.

If that solves the caching of the campaign, it sounds fine to me. I am willing
to implement it to all relevant campaigns if it works. Please contact me when
it is tested in an example campaign.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-06-02 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #14 from Gilles Dubuc gdu...@wikimedia.org ---
Since the Multimedia team is now focusing on UploadWizard for at least 2
quarters, we should find time to implement this feature at some point this
summer, in time for Wiki Loves Monuments.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-05-30 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #13 from Romaine romaine_w...@yahoo.com ---
Can this bug me solved soon or do we have the do nulledits on all the campaigns
at the end of this month and again twice for Wiki Loves Monuments at the begin
and end of September? (about 100 campaigns!!)

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-05-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #9 from Gilles Dubuc gdu...@wikimedia.org ---
You'll need to re-save the template and the changes should trickle down. The
caching introduced at the end of last year was probably introduced as an
operational necessity, I imagine that the lack of caching was inefficient. That
sort of thing is rarely introduced for frivolous reasons, in all likelihood it
means that the old cache-less technique was a strain on servers.

The caching is what's preventing the time-based template from updating
automatically. Re-saving each campaign shouldn't be necessary, just re-saving
the common template should do the job. That sounds like a reasonable workaround
to me. The automatic time-based conditions won't work (since you have to
re-save the template anyway), but at least there's only one template to update
for all campaigns. Thus it scales to any amount of campaigns running at the
same time. I agree that having to re-save all campaigns would be a major
inconvenience and time sink, but it's not necessary (granted, once again, that
the template existed the last time the campaigns were saved).

The issue of having to re-save the template at a specific time remains the only
inconvenience, but it's a side-effect of the caching that I think can't be
worked around easily. I'm not very familiar with templates, could you show me
what rules/syntax are used for the time-based stuff? This way I could
investigate how they work and if there's any chance the campaigns could support
them better. I'm not very hopeful at this point, though, I think it's quite
likely that this limitation is going to be impossible to avoid.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-05-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #10 from vriul...@hotmail.com ---
(In reply to Gilles Dubuc from comment #9)
 You'll need to re-save the template and the changes should trickle down.

I tried to re-save the template but it did not work after Romaine re-saved each
campaign. Probably a campaign should have a definition for starting and ending
time to manage it correctly avoiding a trick with templates.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-05-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #11 from Gilles Dubuc gdu...@wikimedia.org ---
Yes, that sounds like a much better idea. Basically define the start and end
date in the campaign settings, and have optional content/template settings to
display before, during and after. I'll file it as a task.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-05-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #12 from Gilles Dubuc gdu...@wikimedia.org ---
https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/612

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-05-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #7 from Romaine romaine_w...@yahoo.com ---
I do not think this describes the experienced problem. (We do not have a red
link issue.) On top of the campaign, we want to be able to show a banner if the
campaign has not started yet, a banner for if it is running or a banner if it
has ended.

With Wiki Loves Monuments we had about 100 campaigns we had to start. To
prevent errors and to lighten the workload, we introduced in 2012 a timer
template. This timer template automatically changes the banner (before, during
or after) at certain points in time: when the contest starts or ends. This
worked perfectly in both 2012 and 2013. Users see the banner and think: ok, now
my uploads are eligible or not.

Now with Wiki Loves Earth, we use exactly the same set-up and now it doesn't
work any more. When the point of time is reached that a contest starts/ends,
the campaign shows the previous banner template instead of the actual banner.
(Previous it showed the actual banner.)

Only when we did a null edit on every campaign, the right banner was shown.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-05-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #8 from Romaine romaine_w...@yahoo.com ---
This problem of not showing the actual banner caused a lot of confusion at the
beginning of this month and a lot of users reported this problem to us, it is
an essential part of the infrastructure of contests.

Wiki Loves Monuments is coming again in September with many campaigns, also at
the end of this month and a month later this is a problem.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-05-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

Gilles Dubuc gdu...@wikimedia.org changed:

   What|Removed |Added

 CC||gdu...@wikimedia.org
Summary|Upload wizard is cached |Template used as campaign
   |unwantedly  |headerLabel is cached
   ||unwantedly

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-05-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #3 from Gilles Dubuc gdu...@wikimedia.org ---
Confirmed locally. When the template doesn't exist UploadWizard shows a red
link, when the template is created, UW picks it up, but when the template is
updated, UW still displays the old version.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-05-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #4 from Gilles Dubuc gdu...@wikimedia.org ---
Caching was introduced here: https://gerrit.wikimedia.org/r/#/c/86960/

The commit message suggests that there's supposed to be a job-based
invalidation.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-05-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #5 from Gilles Dubuc gdu...@wikimedia.org ---
OK, I think I've figured it out. Basically that caching was introduced and it
has a couple of limitations, which I don't think we can avoid:

The link between the campaign and the templates it uses will only be created if
you create or save the campaign while the template exists. If you create the
campaign before the templates it references, the connection won't be made and
subsequent updates to the template won't get picked up. The workaround is to
re-save the campaign once you've created the template. That create the link
between the two and any subsequent update of the template will update the
campaign.

Secondly, the parent template (in this case
https://commons.wikimedia.org/w/index.php?title=Template:Wiki_Loves_Earth_is_running
) needs to be saved in order for the update to trickle down to the template.
That's a single save to update all the campaigns though. This is what you used
to have, right?

Lastly, the update that trickles down to the campaigns runs through a job, so
it won't be immediate. I don't know what the job queue on Commons is usually
like, but it could be seconds, minutes, or hours depending on the job queue at
the time of the template update.

Does that all make sense? I imagine you either ran into the red link issue
(referencing templates before they exist) or the fact that the update is
asynchronous. As for this caching, I imagine it was introduced for performance
reasons.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 64685] Template used as campaign headerLabel is cached unwantedly

2014-05-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=64685

--- Comment #6 from Gilles Dubuc gdu...@wikimedia.org ---
And by the way I've verified that the update/invalidation works correctly on my
local install.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l