Bug 388168 - networkingclient test application does not recognize state change of networkingclient
Summary: networkingclient test application does not recognize state change of networki...
Status: RESOLVED FIXED
Alias: None
Product: frameworks-kdelibs4support
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 5.41.0
Platform: Other Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: kdelibs bugs
URL:
Keywords:
Depends on: 388727
Blocks: 380136
  Show dependency treegraph
 
Reported: 2017-12-23 09:06 UTC by Ralf Habacker
Modified: 2021-03-04 09:26 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 5.43.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ralf Habacker 2017-12-23 09:06:49 UTC
networkingclient shows state change of the related test app networkingservice. On windows networkingclient does not show the state changes.

This issue may affect other applications depending on kdelibs4support.
Comment 1 Ralf Habacker 2018-01-09 13:56:22 UTC
Running the networkingclient prints out the following message:

[384] Warning:QObject::connect: No such slot solid::NetworkingPrivate::serviceStatusChanged(bool)

which seems to be created by the following code fragment located in src/solid/networking_p.h

#ifndef Q_OS_WIN
    /**
     * Called on DBus signal from the network status service
     */
    void serviceStatusChanged(uint status);
#else
    void serviceStatusChanged(bool status);
#endif

Because moc does not define Q_OS_WIN a signal with uint type is generated and therefore related connection fails -> need to to define Q_OS_WIN on running moc on Windows.
Comment 2 Ralf Habacker 2018-01-10 14:11:23 UTC
(In reply to Ralf Habacker from comment #1)
> Running the networkingclient prints out the following message:
> 
> [384] Warning:QObject::connect: No such slot
> solid::NetworkingPrivate::serviceStatusChanged(bool)

fixed with https://phabricator.kde.org/D9468?vs=24256&id=25088&whitespace=ignore-most#toc
Comment 3 Ralf Habacker 2018-01-11 21:33:23 UTC
See https://phabricator.kde.org/D9468 for the related review request
Comment 4 Christoph Feck 2018-01-31 00:29:55 UTC
Code review from comment #3 says it is committed. Anything else needed for resolving this ticket?
Comment 5 Ralf Habacker 2021-03-04 08:23:34 UTC
(In reply to Ralf Habacker from comment #3)
> See https://phabricator.kde.org/D9468 for the related review request

The patch from the mentioned review was submitted with commit https://phabricator.kde.org/R239:1d1af3e514071efc51281b7ac1154737e44aa224
Comment 6 Ralf Habacker 2021-03-04 08:29:16 UTC
(In reply to Christoph Feck from comment #4)
> Code review from comment #3 says it is committed. Anything else needed for
> resolving this ticket?

I need to check that next time I'm coming back to kdelib4support.