[kopete/Applications/18.08] /: Remove the Yahoo protocol

[Pino Toscano]

Git commit da33bbfb24e3c026ee7eadb5f37085df80bfe481 by Pino Toscano.
Committed on 19/07/2018 at 03:56.
Pushed by pino into branch 'Applications/18.08'.

Remove the Yahoo protocol

Yahoo discontinued the Yahoo Messenger on July 17, 2018, so this
protocol is effectively dead.

As a result:
- remove the code of the protocol itself
- remove all the supporting cmake stuff
- remove one notification specific to it
- remove the kconf_update scripts
- slightly update the documentation to not mention Yahoo anymore
- stop mentioning it in the appdata file
This way, developers/downstreams will not try to accidentally build/ship
this anymore.

BUG: 395287
Fixed-In: 18.08.0

[1] https://help.yahoo.com/kb/SLN28776.html

M  +0-3CMakeLists.txt
M  +1-6doc/index.docbook
M  +2-2kopete/kconf_update/CMakeLists.txt
D  +0-20   kopete/kconf_update/kopete-update_yahoo_server.pl
D  +0-4kopete/kconf_update/kopete-update_yahoo_server.upd
M  +0-137  kopete/kopete.notifyrc
M  +1-1kopete/org.kde.kopete.appdata.xml
M  +0-8libkopete/kopete_export.h
M  +0-4protocols/CMakeLists.txt
D  +0-70   protocols/yahoo/CMakeLists.txt
D  +0-9protocols/yahoo/icons/CMakeLists.txt
D  +---protocols/yahoo/icons/cr16-action-yahoo_connecting.mng
D  +---protocols/yahoo/icons/ox128-app-yahoo_protocol.png
D  +---protocols/yahoo/icons/ox16-action-yahoo_away.png
D  +---protocols/yahoo/icons/ox16-action-yahoo_busy.png
D  +---protocols/yahoo/icons/ox16-action-yahoo_idle.png
D  +---protocols/yahoo/icons/ox16-action-yahoo_invisible.png
D  +---protocols/yahoo/icons/ox16-action-yahoo_mobile.png
D  +---protocols/yahoo/icons/ox16-action-yahoo_stealthed.png
D  +---protocols/yahoo/icons/ox16-action-yahoo_tea.png
D  +---protocols/yahoo/icons/ox16-app-yahoo_protocol.png
D  +---protocols/yahoo/icons/ox22-action-yahoo_stealthed.png
D  +---protocols/yahoo/icons/ox32-action-yahoo_stealthed.png
D  +---protocols/yahoo/icons/ox32-app-yahoo_protocol.png
D  +---protocols/yahoo/icons/ox48-app-yahoo_protocol.png
D  +---protocols/yahoo/icons/ox64-app-yahoo_protocol.png
D  +0-149  protocols/yahoo/kopete_yahoo.desktop
D  +0-91   protocols/yahoo/libkyahoo/CMakeLists.txt
D  +0-46   protocols/yahoo/libkyahoo/alivetask.cpp
D  +0-36   protocols/yahoo/libkyahoo/alivetask.h
D  +0-277  protocols/yahoo/libkyahoo/bytestream.cpp
D  +0-76   protocols/yahoo/libkyahoo/bytestream.h
D  +0-85   protocols/yahoo/libkyahoo/changestatustask.cpp
D  +0-49   protocols/yahoo/libkyahoo/changestatustask.h
D  +0-63   protocols/yahoo/libkyahoo/chatsessiontask.cpp
D  +0-45   protocols/yahoo/libkyahoo/chatsessiontask.h
D  +0-1015 protocols/yahoo/libkyahoo/client.cpp
D  +0-755  protocols/yahoo/libkyahoo/client.h
D  +0-274  protocols/yahoo/libkyahoo/conferencetask.cpp
D  +0-57   protocols/yahoo/libkyahoo/conferencetask.h
D  +0-60   protocols/yahoo/libkyahoo/connector.cpp
D  +0-58   protocols/yahoo/libkyahoo/connector.h
D  +0-214  protocols/yahoo/libkyahoo/coreprotocol.cpp
D  +0-106  protocols/yahoo/libkyahoo/coreprotocol.h
D  +0-202  protocols/yahoo/libkyahoo/crypt.c
D  +0-158  protocols/yahoo/libkyahoo/filetransfernotifiertask.cpp
D  +0-50   protocols/yahoo/libkyahoo/filetransfernotifiertask.h
D  +0-97   protocols/yahoo/libkyahoo/inputprotocolbase.cpp
D  +0-76   protocols/yahoo/libkyahoo/inputprotocolbase.h
D  +0-20   protocols/yahoo/libkyahoo/libkyahoo_export.h
D  +0-529  protocols/yahoo/libkyahoo/libyahoo.c
D  +0-60   protocols/yahoo/libkyahoo/libyahoo.h
D  +0-108  protocols/yahoo/libkyahoo/listtask.cpp
D  +0-46   protocols/yahoo/libkyahoo/listtask.h
D  +0-441  protocols/yahoo/libkyahoo/logintask.cpp
D  +0-95   protocols/yahoo/libkyahoo/logintask.h
D  +0-43   protocols/yahoo/libkyahoo/logofftask.cpp
D  +0-34   protocols/yahoo/libkyahoo/logofftask.h
D  +0-78   protocols/yahoo/libkyahoo/mailnotifiertask.cpp
D  +0-44   protocols/yahoo/libkyahoo/mailnotifiertask.h
D  +0-391  protocols/yahoo/libkyahoo/md5.c
D  +0-93   protocols/yahoo/libkyahoo/md5.h
D  +0-196  protocols/yahoo/libkyahoo/messagereceivertask.cpp
D  +0-50   protocols/yahoo/libkyahoo/messagereceivertask.h
D  +0-165  protocols/yahoo/libkyahoo/modifybuddytask.cpp
D  +0-65   protocols/yahoo/libkyahoo/modifybuddytask.h
D  +0-205  protocols/yahoo/libkyahoo/modifyyabtask.cpp
D  +0-62   protocols/yahoo/libkyahoo/modifyyabtask.h
D  +0-31   protocols/yahoo/libkyahoo/oscartypes.h
D  +0-159  protocols/yahoo/libkyahoo/picturenotifiertask.cpp
D  +0-52   protocols/yahoo/libkyahoo/picturenotifiertask.h
D  +0-45   protocols/yahoo/libkyahoo/pingtask.cpp
D  +0-36   protocols/yahoo/libkyahoo/pingtask.h
D  +0-300  protocols/yahoo/libkyahoo/receivefiletask.cpp
D  +0-85   

D13450: Add arrow keys to move and resize selection rectangle

[Henrik Fehlauer]

rkflx added a comment.


  In D13450#294541 , @sharvey wrote:
  
  > In D13450#294538 , @rkflx wrote:
  >
  > > - Revert unnecessary layout changes.
  >
  >
  > Please clarify... layout for the help text? I don’t want to waste time.
  
  
  I was referring to the following inline comment: D13450#inline-74855 
, e.g. the addition of 
`RowLayout` etc., which should not be necessary if I understand things 
correctly.

REPOSITORY
  R166 Spectacle

REVISION DETAIL
  https://phabricator.kde.org/D13450

To: sharvey, rkflx, ngraham, #spectacle, yurchor
Cc: ltoscano, kde-doc-english, abalaji, #spectacle, skadinna

D13450: Add arrow keys to move and resize selection rectangle

[Scott Harvey]

sharvey added a comment.


  In D13450#294538 , @rkflx wrote:
  
  > Tagging is on Thursday at 23:59 UTC. I will accept the patch tomorrow so 
you can still land it in time (let me know if you are busy and I should commit 
on your behalf).
  
  
  That’ll be 6:59pm in my world. I’ll see what I can accomplish in the next 24 
hours.
  
  > - Revert unnecessary layout changes.
  
  Please clarify... layout for the help text? I don’t want to waste time.
  
  > The rest can be improved upon later.
  
  If it’s not ready (E.g. the edge-touching issue), then it’s not ready. I 
don’t want to see anything released (even as a beta) that’s got bugs. 
Especially known bugs that we just ran out of time to fix. If it has to wait, 
it can wait. I don’t want to be responsible for any bending or breaking of 
rules, and I certainly don’t want to release dodgy software.

REPOSITORY
  R166 Spectacle

REVISION DETAIL
  https://phabricator.kde.org/D13450

To: sharvey, rkflx, ngraham, #spectacle, yurchor
Cc: ltoscano, kde-doc-english, abalaji, #spectacle, skadinna

D13450: Add arrow keys to move and resize selection rectangle

[Henrik Fehlauer]

rkflx added a comment.


  Tagging is on Thursday at 23:59 UTC. I will accept the patch tomorrow so you 
can still land it in time (let me know if you are busy and I should commit on 
your behalf).
  
  It would be great if you could still implement three changes until then:
  
  - Move one line in the help text around (see inline comment).
  - Fix [Alt] + [↓].
  - Revert unnecessary layout changes.
  
  The rest can be improved upon later.

REPOSITORY
  R166 Spectacle

REVISION DETAIL
  https://phabricator.kde.org/D13450

To: sharvey, rkflx, ngraham, #spectacle, yurchor
Cc: ltoscano, kde-doc-english, abalaji, #spectacle, skadinna

D13450: Add arrow keys to move and resize selection rectangle

[Henrik Fehlauer]

rkflx added a comment.


  Thanks for the updates. Found one more problem, the other changes are fine.
  
  In D13450#294047 , @rkflx wrote:
  
  > - When the rectangle covers the whole screen, I'm unable to resize, i.e. 
make the rectangle smaller again.
  
  
  Note that this problem is more general: Once an edge of the selection touches 
the border of the screen, decreasing the size in that direction is blocked, 
only increasing the size works.
  
  In D13450#294068 , @sharvey wrote:
  
  > In D13450#294047 , @rkflx wrote:
  >
  > > - Selecting with the mouse sets a minimum size, however resizing with the 
arrow keys allows to resize to 0x0px.
  >
  >
  > Where should it stop? 1x1?
  
  
  It should be consistent with how it works for selecting with the mouse (grep 
for `minRectSize`).
  
  In D13450#294279 , @sharvey wrote:
  
  > - Change default adjustment to `largeChange`, [⇧] uses `smallChange` for 
fine tuning
  
  
  Thanks, appreciated!
  
  In D13450#294291 , @ngraham wrote:
  
  > I'm noticing that the magnifier doesn't er show up while you're using the 
keyboard to resize the selection rect. Might wanna hook that up.
  
  
  Yeah, I also had that idea a couple of weeks back, but decided to not bring 
it up in the review, because for conceptual reasons I doubt that's so easy to 
implement like you make it sound: The magnifier shows up at the cursor 
position, which for the mouse case is either at one of the corners, or at an 
arbitrary position along one of the edges. Since with the arrow keys you can 
only control horizontal/vertical movement of the edges (this is true both for 
resizing and moving), there is no well-defined corner, i.e. it does not make 
any sense to show a square magnifier at a random position on the edge. You'd 
have to show a magnifier covering the complete width/height of the respective 
edge, because you cannot know where the user is looking at and which part of 
the screenshot he wants to align the selection to (it might not be the corner).
  
  One way forward could be to not only show the magnifier when resizing (i.e. 
clicking on a handle) as it is implemented currently, but in addition show the 
magnifier directly attached to the cursor as soon as Shift is hold down, e.g. 
more like KWin's magnifying glass. However, this has a couple of drawbacks: In 
case you checked the checkbox to show the magnifier by default, Shift will 
confusingly trigger hiding the magnifier for clicking, but trigger showing when 
not clicking (because without any click or modifier the magnifier should never 
be shown at all). Next, for resizing by keyboard you'd have to first 
move/resize the selection rectangle, and then reach for the mouse to position 
the magnifier where you want it, and iterate as needed. That's pretty awkward, 
if you ask me.
  
  @ngraham Feel free to send a follow-up patch if you can get it working in a 
non-confusing way and have a convincing answer to the question where to show 
the magnifier (or rather what to magnify).
  
  ---
  
  In D13450#294305 , @sharvey wrote:
  
  > You suggested multiple changes to the `checkBounds` function, all of which 
I understand and agree with. This function caused me the most heartburn and I 
feel uncomfortable attempting to rework it while under a time crunch. 
Unfortunately, this leaves the `0x0px` resize and the "cannot resize when 
expanded to full screen` bugs unresolved.
  
  
  Indeed, those are not critical and can be worked on for the RC (although 
that's a huge responsibility, as normally you are not supposed to ship broken 
stuff where the fix is only promised for the future…).
  
  > (Again, thank you for the code optimization suggestions. They are 
appreciated.)
  
  Glad you find them useful. It's not always easy to find the correct 
abstractions to get concise, readable and maintainable code, but practice makes 
perfect ;)

INLINE COMMENTS

> EditorRoot.qml:169
> + case Qt.Key_Down:
> + change = checkBounds(0.0, smallChange, "down");
> + selection.height = selection.height + change;

[Alt] + [↓] is broken (should be fast, not slow)

> rkflx wrote in EditorRoot.qml:92
> Insert space after `switch`, please. Repeat for all other occurrences.

Not done, but maybe I should have given you an example. This is how I meant it:

  switch (event.modifiers) {

IOW, this should be consistent with how the spacing works for `if`.

> rkflx wrote in EditorRoot.qml:173
> Insert space after `case`.

Not yet done.

> rkflx wrote in EditorRoot.qml:443-507
> Currently I don't understand what all those changes are for and how they 
> relate to the topic of the patch (and I would disagree with some).
> 
> Contrary to @ngraham 

D13450: Add arrow keys to move and resize selection rectangle

[Scott Harvey]

sharvey added a comment.


  In D13450#294304 , @ngraham wrote:
  
  > Yeah, the magnifier shows up while dragging the selection rect (or not, 
depending on whether or not you've holding down [⇧]), but it currently doesn't 
show up when you're using [Alt] to resize the rect using the keyboard arrow 
keys.
  >
  > IMHO, if you have the magnifier turned on in the settings, it should show 
up when you're in "fine tune" mode (holding down both [⇧] and [Alt] and using 
the arrow keys to resize the rect by single pixels), because conceptually, if 
you're doing this, it's because you want a pixel-perfect rect.
  
  
  Let's get @rkflx 's input on this. I'm not sure how easily it will be to 
implement. There's a deadline hanging over our heads.

REPOSITORY
  R166 Spectacle

REVISION DETAIL
  https://phabricator.kde.org/D13450

To: sharvey, rkflx, ngraham, #spectacle, yurchor
Cc: ltoscano, kde-doc-english, abalaji, #spectacle, skadinna

D13450: Add arrow keys to move and resize selection rectangle

[Scott Harvey]

sharvey added a comment.


  In D13450#294047 , @rkflx wrote:
  
  > I still disagree regarding the default speed selection. We determined by 
looking at other apps that [⇧] is the modifier to use, and I argued (in line 
with what I meant when triaging the bug) that for making the rectangle feature 
useful for keyboard users by default the movement should be fast. BTW, this is 
also what KWin is doing, and I see no reason at all why Spectacle should 
deviate from that standard. (See above for even more arguments.) I don't feel 
comfortable approving the current default, sorry.
  
  
  Fixed. The default movement - [Arrows] alone - is `largeChange`, and [⇧] + 
[Arrows] is a single pixel move.
  
  > To summarize, here's what I propose: If we can reach agreement on the 
default and you are fast with the string changes still required, the patch can 
go in for the Beta. Then you have time to work on the other points until the RC 
(at which point we have to decide whether to ship or to revert again, based on 
your progress)
  
  The help text box has been reformatted to match the prior layout. `Esc` has 
been moved to the bottom of the list. Also, the docbook entry for this patch 
has been rewritten and is hopefully clearer and more consistent.
  
  ---
  
  You suggested multiple changes to the `checkBounds` function, all of which I 
understand and agree with. This function caused me the most heartburn and I 
feel uncomfortable attempting to rework it while under a time crunch. 
Unfortunately, this leaves the `0x0px` resize and the "cannot resize when 
expanded to full screen` bugs unresolved.

REPOSITORY
  R166 Spectacle

REVISION DETAIL
  https://phabricator.kde.org/D13450

To: sharvey, rkflx, ngraham, #spectacle, yurchor
Cc: ltoscano, kde-doc-english, abalaji, #spectacle, skadinna

D13450: Add arrow keys to move and resize selection rectangle

[Nathaniel Graham]

ngraham added a comment.


  Yeah, the magnifier shows up while dragging the selection rect (or not, 
depending on whether or not you've holding down [⇧]), but it currently doesn't 
show up when you're using [Alt] to resize the rect using the keyboard arrow 
keys.
  
  IMHO, if you have the magnifier turned on in the settings, it should show up 
when you're in "fine tune" mode (holding down both [⇧] and [Alt] and using the 
arrow keys to resize the rect by single pixels), because conceptually, if 
you're doing this, it's because you want a pixel-perfect rect.

REPOSITORY
  R166 Spectacle

REVISION DETAIL
  https://phabricator.kde.org/D13450

To: sharvey, rkflx, ngraham, #spectacle, yurchor
Cc: ltoscano, kde-doc-english, abalaji, #spectacle, skadinna

D13450: Add arrow keys to move and resize selection rectangle

[Scott Harvey]

sharvey added a comment.


  In D13450#294291 , @ngraham wrote:
  
  > I'm noticing that the magnifier doesn't er show up while you're using the 
keyboard to resize the selection rect. Might wanna hook that up.
  
  
  It works for me when holding [⇧] and dragging a new rectangle. I'm a little 
fuzzy on how the magnifier is intended to work.
  
  We've already commandeers [⇧] for small adjustments, so that's likely 
overriding the magnifier.
  
  How do we think this should work?

REPOSITORY
  R166 Spectacle

REVISION DETAIL
  https://phabricator.kde.org/D13450

To: sharvey, rkflx, ngraham, #spectacle, yurchor
Cc: ltoscano, kde-doc-english, abalaji, #spectacle, skadinna

D13450: Add arrow keys to move and resize selection rectangle

[Nathaniel Graham]

ngraham added a comment.


  I'm noticing that the magnifier doesn't er show up while you're using the 
keyboard to resize the selection rect. Might wanna hook that up.

REPOSITORY
  R166 Spectacle

REVISION DETAIL
  https://phabricator.kde.org/D13450

To: sharvey, rkflx, ngraham, #spectacle, yurchor
Cc: ltoscano, kde-doc-english, abalaji, #spectacle, skadinna

D13450: Add arrow keys to move and resize selection rectangle

[Scott Harvey]

sharvey marked 7 inline comments as done.

REPOSITORY
  R166 Spectacle

REVISION DETAIL
  https://phabricator.kde.org/D13450

To: sharvey, rkflx, ngraham, #spectacle, yurchor
Cc: ltoscano, kde-doc-english, abalaji, #spectacle, skadinna

D13450: Add arrow keys to move and resize selection rectangle

[Scott Harvey]

sharvey edited the summary of this revision.
sharvey edited the test plan for this revision.

REPOSITORY
  R166 Spectacle

REVISION DETAIL
  https://phabricator.kde.org/D13450

To: sharvey, rkflx, ngraham, #spectacle, yurchor
Cc: ltoscano, kde-doc-english, abalaji, #spectacle, skadinna

D13450: Add arrow keys to move and resize selection rectangle

[Scott Harvey]

sharvey marked 9 inline comments as done.
sharvey added inline comments.

INLINE COMMENTS

> rkflx wrote in index.docbook:154
> "Holding the Shift while": Either remove "the", or add "key".
> 
> "using the arrows": Change to "Using the arrow keys"?

Typos and/error sloppy grammar. Rewritten.

REPOSITORY
  R166 Spectacle

REVISION DETAIL
  https://phabricator.kde.org/D13450

To: sharvey, rkflx, ngraham, #spectacle, yurchor
Cc: ltoscano, kde-doc-english, abalaji, #spectacle, skadinna

D13450: Add arrow keys to move and resize selection rectangle

[Scott Harvey]

sharvey updated this revision to Diff 38014.
sharvey marked an inline comment as done.
sharvey added a comment.


  - Change default adjustment to `largeChange`, [⇧] uses `smallChange` for fine 
tuning
  - Numerous miscellaneous changes per reviewer comments, including:
- Condensed adjustment calculations to use `+=`
- Reverted help text box to match original layout
- Revised docbook entry for clarity and consistency

REPOSITORY
  R166 Spectacle

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D13450?vs=37965=38014

BRANCH
  arcpatch-D13450

REVISION DETAIL
  https://phabricator.kde.org/D13450

AFFECTED FILES
  doc/index.docbook
  src/QuickEditor/EditorRoot.qml

To: sharvey, rkflx, ngraham, #spectacle, yurchor
Cc: ltoscano, kde-doc-english, abalaji, #spectacle, skadinna

D13450: Add arrow keys to move and resize selection rectangle

[Scott Harvey]

sharvey added inline comments.

INLINE COMMENTS

> rkflx wrote in EditorRoot.qml:443-507
> Currently I don't understand what all those changes are for and how they 
> relate to the topic of the patch (and I would disagree with some).
> 
> Contrary to @ngraham I'd say this actually is a visual regression.

These are remnants (which will be reverted) from when the help text box was two 
columns wide.

REPOSITORY
  R166 Spectacle

REVISION DETAIL
  https://phabricator.kde.org/D13450

To: sharvey, rkflx, ngraham, #spectacle, yurchor
Cc: ltoscano, kde-doc-english, abalaji, #spectacle, skadinna

D13450: Add arrow keys to move and resize selection rectangle

[Scott Harvey]

sharvey added inline comments.

INLINE COMMENTS

> rkflx wrote in EditorRoot.qml:41
> That does not seem right, because this will result in slower movement on a 
> HiDPI screen compared to a regular screen of the same size. (It's needed for 
> the 1px case, of course.)

This was suggested by @abalaji based on his work on reworking the selection 
rectangle. Should it be removed?

REPOSITORY
  R166 Spectacle

REVISION DETAIL
  https://phabricator.kde.org/D13450

To: sharvey, rkflx, ngraham, #spectacle, yurchor
Cc: ltoscano, kde-doc-english, abalaji, #spectacle, skadinna