[Bug 33260] WikiLove posts above LiquidThreads rather than within a thread

2013-09-11 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33260

Helder  changed:

   What|Removed |Added

   See Also||https://bugzilla.wikimedia.
   ||org/show_bug.cgi?id=52729

-- 
You are receiving this mail because:
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 33260] WikiLove posts above LiquidThreads rather than within a thread

2013-09-11 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33260

Helder  changed:

   What|Removed |Added

   See Also||https://bugzilla.wikimedia.
   ||org/show_bug.cgi?id=41276

-- 
You are receiving this mail because:
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 33260] WikiLove posts above LiquidThreads rather than within a thread

2013-09-10 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33260

Legoktm  changed:

   What|Removed |Added

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

-- 
You are receiving this mail because:
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 33260] WikiLove posts above LiquidThreads rather than within a thread

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

--- Comment #13 from Gerrit Notification Bot  ---
Change 83409 merged by jenkins-bot:
Create new LQT thread if possible

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

-- 
You are receiving this mail because:
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 33260] WikiLove posts above LiquidThreads rather than within a thread

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

Legoktm  changed:

   What|Removed |Added

 CC||legoktm.wikipe...@gmail.com
 Depends on|26312   |
   Assignee|wikibugs-l@lists.wikimedia. |legoktm.wikipe...@gmail.com
   |org |

--- Comment #12 from Legoktm  ---
(In reply to comment #6)
> Looks like Krinkle beat me to it and filed the bug last year:
> https://bugzilla.wikimedia.org/show_bug.cgi?id=26312
> 
> I'll mark it as a dependency.
> 

What does this have to do with LiquidThreads at all? I've removed it as a
dependency.

-- 
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 33260] WikiLove posts above LiquidThreads rather than within a thread

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

Gerrit Notification Bot  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 33260] WikiLove posts above LiquidThreads rather than within a thread

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

--- Comment #11 from Gerrit Notification Bot  ---
Change 83409 had a related patch set uploaded by Legoktm:
Create new LQT thread if possible

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

-- 
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 33260] WikiLove posts above LiquidThreads rather than within a thread

2012-05-31 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33260

--- Comment #10 from Thorncrag  2012-05-31 18:08:16 UTC ---
Just as a side note, MZMcBride directed me toward this bug, because I found
that a side-affect of this bug is that users who receive WikiLove when their
user talk page has LQT enabled do not receive a new message notification.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33260] WikiLove posts above LiquidThreads rather than within a thread

2012-05-31 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33260

--- Comment #9 from MZMcBride  2012-05-31 18:05:36 UTC ---
I believe Werdna was rather adamant about &action=edit§ion=new not being
equivalent to posting a LiquidThreads thread. He sees the actions as distinct
(cf. bug 21256 comment 3).

I thought WikiLove was using the MediaWiki API internally, so I was going to
suggest simply checking the target talk page for LiquidThreads and then using
the LiquidThreads action instead of &action=edit§ion=new, however it looks
like WikiLove isn't using the MediaWiki API (for whatever reason).

I glanced at r106754 and it feels very hackish and bad. Particularly these two
lines:

$this->sectiontitle = $wgLang->truncate( $request->getText( 'wpSectionTitle' ),
250 );
$this->sectiontitle = preg_replace( '/^\s*=+\s*(.*?)\s*=+\s*$/', '$1',
$this->sectiontitle );

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33260] WikiLove posts above LiquidThreads rather than within a thread

2011-12-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33260

Mark A. Hershberger  changed:

   What|Removed |Added

   Priority|Unprioritized   |Normal
 CC||m...@everybody.org

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33260] WikiLove posts above LiquidThreads rather than within a thread

2011-12-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33260

--- Comment #8 from Roan Kattouw  2011-12-20 14:34:12 
UTC ---
(In reply to comment #2)
> Instead of appendtext, it should probably use text=...§ion=new, like the
> front-end. That would be closer to something that canonically means wanting to
> open a new discussion when used on a talk page.
> 
I think that LQT actually hijacks section=new for adding a new thread, but I'm
not sure.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33260] WikiLove posts above LiquidThreads rather than within a thread

2011-12-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33260

--- Comment #7 from Ryan Kaldari  2011-12-20 04:26:09 
UTC ---
I've tried to implement a workaround in r106754. I'll leave this open until the
rev is approved.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33260] WikiLove posts above LiquidThreads rather than within a thread

2011-12-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33260

Ryan Kaldari  changed:

   What|Removed |Added

 Depends on||26312

--- Comment #6 from Ryan Kaldari  2011-12-19 23:34:53 
UTC ---
Looks like Krinkle beat me to it and filed the bug last year:
https://bugzilla.wikimedia.org/show_bug.cgi?id=26312

I'll mark it as a dependency.

It looks like the essential problem is that action=edit relies on EditPage.php,
which for some reason reuses the wpSummary input for both the edit summary and
the section title, even though these should be separate inputs.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33260] WikiLove posts above LiquidThreads rather than within a thread

2011-12-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33260

--- Comment #5 from Ryan Kaldari  2011-12-19 23:06:04 
UTC ---
I'm going to file a new bug explaining that magically morphing the summary from
an edit summary into a section title and then auto-generating an edit summary
that is different than the summary makes no sense.

I bet you a dollar it will be closed within 24 hours as WONTFIX since it is
written into the 10 Commandments "Thou shalt not change the MediaWiki API" :P

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33260] WikiLove posts above LiquidThreads rather than within a thread

2011-12-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33260

--- Comment #4 from Ryan Kaldari  2011-12-19 22:53:45 
UTC ---
Actually that's only half true. You can specify an edit summary, but it doesn't
respect what you specify. Instead it forces it into a "new section" message. We
just need a way to override this.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33260] WikiLove posts above LiquidThreads rather than within a thread

2011-12-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33260

Ryan Kaldari  changed:

   What|Removed |Added

  Component|API |WikiLove
Version|unspecified |any
Product|MediaWiki   |MediaWiki extensions

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33260] WikiLove posts above LiquidThreads rather than within a thread

2011-12-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33260

--- Comment #3 from Ryan Kaldari  2011-12-19 22:40:58 
UTC ---
We can't use section=new since that method doesn't allow us to specify the edit
summary.

If someone fixes that, I'll gladly re-implement it.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33260] WikiLove posts above LiquidThreads rather than within a thread

2011-12-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33260

Krinkle  changed:

   What|Removed |Added

 CC||krinklem...@gmail.com

--- Comment #2 from Krinkle  2011-12-19 22:20:55 UTC ---
Hm.. it would be an API bug if it the canonical goal behind appending text to a
page is to create a new thread, or even to participate in discussion at all.

However, for one, the header of an LQT-enabled page should be editable as well.
And secondly, how can we distinguish from appending something to the last
section and appending a new section ?

Instead of appendtext, it should probably use text=...§ion=new, like the
front-end. That would be closer to something that canonically means wanting to
open a new discussion when used on a talk page.

Theoratically a bot could be using text=..§ion=new to create a new section
in the header of an LQT-enabled page, so we'd have to find a way to do that
still.

Perhaps MediaWiki should get an additional parameter (or different
action/module alltogether) for participating in discussions and letting
extensions override this ? By default it would implement a 'raw wikitext'
discussion thread, which is just sections on a talk page.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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 33260] WikiLove posts above LiquidThreads rather than within a thread

2011-12-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=33260

Ryan Kaldari  changed:

   What|Removed |Added

 CC||bryan.tongm...@gmail.com,
   ||roan.katt...@gmail.com,
   ||s...@reedyboy.net,
   ||soxre...@gmail.com
  Component|WikiLove|API
Version|any |unspecified
Product|MediaWiki extensions|MediaWiki

--- Comment #1 from Ryan Kaldari  2011-12-19 22:14:42 
UTC ---
WikiLove just uses the MediaWiki API to post the message (via the appendtext
parameter using action=edit). So this is actually an API bug.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- 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