Merge lp:~nik90/ubuntu-clock-app/backend-plugins-rename into lp:ubuntu-clock-app

Proposed by Nekhelesh Ramananthan
Status: Merged
Approved by: Bartosz Kosiorek
Approved revision: 369
Merged at revision: 363
Proposed branch: lp:~nik90/ubuntu-clock-app/backend-plugins-rename
Merge into: lp:ubuntu-clock-app
Diff against target: 917 lines (+106/-378)
28 files modified
app/alarm/AlarmSettingsPage.qml (+2/-2)
app/alarm/AlarmSound.qml (+1/-1)
app/alarm/EditAlarmPage.qml (+2/-2)
app/clock/ClockPage.qml (+1/-1)
app/ubuntu-clock-app.qml (+2/-2)
app/worldclock/UserWorldCityList.qml (+1/-1)
app/worldclock/WorldCityList.qml (+1/-1)
backend/CMakeLists.txt (+41/-107)
backend/modules/Alarm/backend.cpp (+5/-3)
backend/modules/Alarm/qmldir (+2/-2)
backend/modules/Alarm/settings.cpp (+13/-13)
backend/modules/Alarm/settings.h (+4/-4)
backend/modules/Alarm/sound.cpp (+10/-10)
backend/modules/Alarm/sound.h (+4/-4)
backend/modules/DateTime/backend.cpp (+0/-34)
backend/modules/DateTime/backend.h (+0/-35)
backend/modules/DateTime/qmldir (+0/-2)
backend/modules/GeoLocation/backend.cpp (+0/-34)
backend/modules/GeoLocation/backend.h (+0/-35)
backend/modules/GeoLocation/qmldir (+0/-2)
backend/modules/Timezone/backend.cpp (+0/-39)
backend/modules/Timezone/backend.h (+0/-35)
backend/modules/Timezone/qmldir (+0/-2)
backend/modules/WorldClock/backend.cpp (+12/-3)
backend/modules/WorldClock/qmldir (+2/-2)
debian/changelog (+1/-0)
tests/unit/MockClockApp.qml (+1/-1)
tests/unit/tst_alarm.qml (+1/-1)
To merge this branch: bzr merge lp:~nik90/ubuntu-clock-app/backend-plugins-rename
Reviewer Review Type Date Requested Status
Bartosz Kosiorek Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Nekhelesh Ramananthan Abstain
Review via email: mp+269089@code.launchpad.net

Commit message

Renamed C++ plugins and grouped them by functionality. We have 3 main plugins Clock, Alarm and Stopwatch. Each plugin has its own classes like TimeZone, DateTime, LapHistory etc.

Description of the change

Rename backend plugins as we discussed

To post a comment you must log in.
Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

Still need to add entry to debian changelog. However I just wanted to get your opinion on this.

review: Needs Fixing
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
366. By Nekhelesh Ramananthan

renamed Clock to WorldClock

367. By Nekhelesh Ramananthan

Renamed Settings to AlarmSettings and also Sound to AlarmSound

368. By Nekhelesh Ramananthan

merged trunk

369. By Nekhelesh Ramananthan

Updated debian changelog

Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

Fixed the initial set of comments mentioned by bartosz on g+. Also updated debian changelog

review: Abstain
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Bartosz Kosiorek (gang65) wrote :

Nice job.

One small nitpick: Instead of Settings::Settings it should be AlarmSettings::AlarmSettings.
We decided that it is more clear for developers.

review: Needs Fixing
Revision history for this message
Bartosz Kosiorek (gang65) wrote :

Everything is fine

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'app/alarm/AlarmSettingsPage.qml'
--- app/alarm/AlarmSettingsPage.qml 2015-08-14 05:34:49 +0000
+++ app/alarm/AlarmSettingsPage.qml 2015-08-26 23:37:55 +0000
@@ -18,8 +18,8 @@
1818
19import QtQuick 2.419import QtQuick 2.4
20import QtQuick.Layouts 1.120import QtQuick.Layouts 1.1
21import DateTime 1.021import WorldClock 1.0
22import Alarm.Settings 1.022import Alarm 1.0
23import Ubuntu.Components 1.223import Ubuntu.Components 1.2
24import "../components"24import "../components"
2525
2626
=== modified file 'app/alarm/AlarmSound.qml'
--- app/alarm/AlarmSound.qml 2015-08-24 15:08:21 +0000
+++ app/alarm/AlarmSound.qml 2015-08-26 23:37:55 +0000
@@ -17,7 +17,7 @@
17 */17 */
1818
19import QtQuick 2.419import QtQuick 2.4
20import Clock.Utility 1.020import Alarm 1.0
21import QtMultimedia 5.021import QtMultimedia 5.0
22import Ubuntu.Content 1.122import Ubuntu.Content 1.1
23import Ubuntu.Components 1.223import Ubuntu.Components 1.2
2424
=== modified file 'app/alarm/EditAlarmPage.qml'
--- app/alarm/EditAlarmPage.qml 2015-08-25 11:39:19 +0000
+++ app/alarm/EditAlarmPage.qml 2015-08-26 23:37:55 +0000
@@ -17,8 +17,8 @@
17 */17 */
1818
19import QtQuick 2.419import QtQuick 2.4
20import DateTime 1.020import WorldClock 1.0
21import Clock.Utility 1.021import Alarm 1.0
22import Ubuntu.Components 1.222import Ubuntu.Components 1.2
23import Qt.labs.folderlistmodel 2.123import Qt.labs.folderlistmodel 2.1
24import Ubuntu.Components.Pickers 1.024import Ubuntu.Components.Pickers 1.0
2525
=== modified file 'app/clock/ClockPage.qml'
--- app/clock/ClockPage.qml 2015-08-17 14:09:09 +0000
+++ app/clock/ClockPage.qml 2015-08-26 23:37:55 +0000
@@ -20,7 +20,7 @@
20import U1db 1.0 as U1db20import U1db 1.0 as U1db
21import QtPositioning 5.221import QtPositioning 5.2
22import Ubuntu.Components 1.222import Ubuntu.Components 1.2
23import GeoLocation 1.023import WorldClock 1.0
24import "../components"24import "../components"
25import "../worldclock"25import "../worldclock"
2626
2727
=== modified file 'app/ubuntu-clock-app.qml'
--- app/ubuntu-clock-app.qml 2015-08-24 15:08:21 +0000
+++ app/ubuntu-clock-app.qml 2015-08-26 23:37:55 +0000
@@ -17,9 +17,9 @@
17 */17 */
1818
19import QtQuick 2.419import QtQuick 2.4
20import DateTime 1.020import WorldClock 1.0
21import U1db 1.0 as U1db21import U1db 1.0 as U1db
22import Clock.Utility 1.022import Alarm 1.0
23import Ubuntu.Components 1.223import Ubuntu.Components 1.2
24import "components"24import "components"
2525
2626
=== modified file 'app/worldclock/UserWorldCityList.qml'
--- app/worldclock/UserWorldCityList.qml 2015-08-17 12:27:20 +0000
+++ app/worldclock/UserWorldCityList.qml 2015-08-26 23:37:55 +0000
@@ -17,7 +17,7 @@
17 */17 */
1818
19import QtQuick 2.419import QtQuick 2.4
20import Timezone 1.020import WorldClock 1.0
21import U1db 1.0 as U1db21import U1db 1.0 as U1db
22import Ubuntu.Components 1.222import Ubuntu.Components 1.2
2323
2424
=== modified file 'app/worldclock/WorldCityList.qml'
--- app/worldclock/WorldCityList.qml 2015-08-14 05:34:49 +0000
+++ app/worldclock/WorldCityList.qml 2015-08-26 23:37:55 +0000
@@ -17,7 +17,7 @@
17 */17 */
1818
19import QtQuick 2.419import QtQuick 2.4
20import Timezone 1.020import WorldClock 1.0
21import Ubuntu.Components 1.221import Ubuntu.Components 1.2
22import "../components"22import "../components"
23import "../upstreamcomponents"23import "../upstreamcomponents"
2424
=== modified file 'backend/CMakeLists.txt'
--- backend/CMakeLists.txt 2015-08-26 22:41:34 +0000
+++ backend/CMakeLists.txt 2015-08-26 23:37:55 +0000
@@ -8,36 +8,21 @@
8)8)
99
10set(10set(
11 timezone_SRCS11 alarm_SRCS
12 modules/Timezone/backend.cpp12 modules/Alarm/backend.cpp
13 modules/Timezone/timezonemodel.cpp13 modules/Alarm/settings.cpp
14 modules/Timezone/generictimezonemodel.cpp14 modules/Alarm/sound.cpp
15 modules/Timezone/jsontimezonemodel.cpp15)
16 modules/Timezone/statictimezonemodel.cpp16
17)17set(
1818 worldclock_SRCS
19set(19 modules/WorldClock/backend.cpp
20 datetime_SRCS20 modules/WorldClock/geolocation.cpp
21 modules/DateTime/backend.cpp21 modules/WorldClock/datetime.cpp
22 modules/DateTime/datetime.cpp22 modules/WorldClock/timezonemodel.cpp
23)23 modules/WorldClock/generictimezonemodel.cpp
2424 modules/WorldClock/jsontimezonemodel.cpp
25set(25 modules/WorldClock/statictimezonemodel.cpp
26 alarmsettings_SRCS
27 modules/Alarm/Settings/backend.cpp
28 modules/Alarm/Settings/alarmsettings.cpp
29)
30
31set(
32 geolocation_SRCS
33 modules/GeoLocation/backend.cpp
34 modules/GeoLocation/geolocation.cpp
35)
36
37set(
38 clockutility_SRCS
39 modules/Clock/Utility/backend.cpp
40 modules/Clock/Utility/alarmsound.cpp
41)26)
4227
43set(28set(
@@ -47,84 +32,42 @@
47 modules/Stopwatch/formattime.cpp32 modules/Stopwatch/formattime.cpp
48)33)
4934
50add_library(timezone MODULE35add_library(alarm MODULE
51 ${timezone_SRCS}36 ${alarm_SRCS}
52)37)
5338
54add_library(datetime MODULE39add_library(worldclock MODULE
55 ${datetime_SRCS}40 ${worldclock_SRCS}
56)
57
58add_library(alarmsettings MODULE
59 ${alarmsettings_SRCS}
60)
61
62add_library(geolocation MODULE
63 ${geolocation_SRCS}
64)
65
66add_library(clockutility MODULE
67 ${clockutility_SRCS}
68)41)
6942
70add_library(stopwatch MODULE43add_library(stopwatch MODULE
71 ${stopwatch_SRCS}44 ${stopwatch_SRCS}
72)45)
7346
74set_target_properties(timezone PROPERTIES47set_target_properties(alarm PROPERTIES
75 LIBRARY_OUTPUT_DIRECTORY Timezone48 LIBRARY_OUTPUT_DIRECTORY Alarm
76)49)
7750
78set_target_properties(datetime PROPERTIES51set_target_properties(worldclock PROPERTIES
79 LIBRARY_OUTPUT_DIRECTORY DateTime52 LIBRARY_OUTPUT_DIRECTORY WorldClock
80)
81
82set_target_properties(alarmsettings PROPERTIES
83 LIBRARY_OUTPUT_DIRECTORY Alarm/Settings
84)
85
86set_target_properties(geolocation PROPERTIES
87 LIBRARY_OUTPUT_DIRECTORY GeoLocation
88)
89
90set_target_properties(clockutility PROPERTIES
91 LIBRARY_OUTPUT_DIRECTORY Clock/Utility
92)53)
9354
94set_target_properties(stopwatch PROPERTIES55set_target_properties(stopwatch PROPERTIES
95 LIBRARY_OUTPUT_DIRECTORY Stopwatch/56 LIBRARY_OUTPUT_DIRECTORY Stopwatch/
96)57)
9758
98qt5_use_modules(datetime Gui Qml Quick)59qt5_use_modules(alarm Gui Qml Quick DBus)
99qt5_use_modules(timezone Gui Qml Quick)60qt5_use_modules(worldclock Gui Qml Quick)
100qt5_use_modules(alarmsettings Gui Qml Quick DBus)
101qt5_use_modules(geolocation Gui Qml Quick)
102qt5_use_modules(clockutility Qml)
103qt5_use_modules(stopwatch Qml)61qt5_use_modules(stopwatch Qml)
10462
105# Copy qmldir file to build dir for running in QtCreator63# Copy qmldir file to build dir for running in QtCreator
106add_custom_target(timezone-qmldir ALL64add_custom_target(alarm-qmldir ALL
107 COMMAND ${CMAKE_COMMAND} -E copy ${CMAKE_CURRENT_SOURCE_DIR}/modules/Timezone/qmldir ${CMAKE_CURRENT_BINARY_DIR}/Timezone65 COMMAND ${CMAKE_COMMAND} -E copy ${CMAKE_CURRENT_SOURCE_DIR}/modules/Alarm/qmldir ${CMAKE_CURRENT_BINARY_DIR}/Alarm
108 DEPENDS ${QMLFILES}66 DEPENDS ${QMLFILES}
109)67)
11068
111add_custom_target(datetime-qmldir ALL69add_custom_target(worldclock-qmldir ALL
112 COMMAND ${CMAKE_COMMAND} -E copy ${CMAKE_CURRENT_SOURCE_DIR}/modules/DateTime/qmldir ${CMAKE_CURRENT_BINARY_DIR}/DateTime70 COMMAND ${CMAKE_COMMAND} -E copy ${CMAKE_CURRENT_SOURCE_DIR}/modules/WorldClock/qmldir ${CMAKE_CURRENT_BINARY_DIR}/WorldClock
113 DEPENDS ${QMLFILES}
114)
115
116add_custom_target(alarmsettings-qmldir ALL
117 COMMAND ${CMAKE_COMMAND} -E copy ${CMAKE_CURRENT_SOURCE_DIR}/modules/Alarm/Settings/qmldir ${CMAKE_CURRENT_BINARY_DIR}/Alarm/Settings
118 DEPENDS ${QMLFILES}
119)
120
121add_custom_target(geolocation-qmldir ALL
122 COMMAND ${CMAKE_COMMAND} -E copy ${CMAKE_CURRENT_SOURCE_DIR}/modules/GeoLocation/qmldir ${CMAKE_CURRENT_BINARY_DIR}/GeoLocation
123 DEPENDS ${QMLFILES}
124)
125
126add_custom_target(clockutility-qmldir ALL
127 COMMAND ${CMAKE_COMMAND} -E copy ${CMAKE_CURRENT_SOURCE_DIR}/modules/Clock/Utility/qmldir ${CMAKE_CURRENT_BINARY_DIR}/Clock/Utility
128 DEPENDS ${QMLFILES}71 DEPENDS ${QMLFILES}
129)72)
13073
@@ -134,20 +77,11 @@
134)77)
13578
136# Install plugin file79# Install plugin file
137install(TARGETS timezone DESTINATION ${MODULE_PATH}/Timezone/)80install(TARGETS alarm DESTINATION ${MODULE_PATH}/Alarm/)
138install(FILES modules/Timezone/qmldir DESTINATION ${MODULE_PATH}/Timezone/)81install(FILES modules/Alarm/qmldir DESTINATION ${MODULE_PATH}/Alarm/)
13982
140install(TARGETS datetime DESTINATION ${MODULE_PATH}/DateTime/)83install(TARGETS worldclock DESTINATION ${MODULE_PATH}/WorldClock/)
141install(FILES modules/DateTime/qmldir DESTINATION ${MODULE_PATH}/DateTime/)84install(FILES modules/WorldClock/qmldir DESTINATION ${MODULE_PATH}/WorldClock/)
142
143install(TARGETS alarmsettings DESTINATION ${MODULE_PATH}/Alarm/Settings/)
144install(FILES modules/Alarm/Settings/qmldir DESTINATION ${MODULE_PATH}/Alarm/Settings/)
145
146install(TARGETS geolocation DESTINATION ${MODULE_PATH}/GeoLocation/)
147install(FILES modules/GeoLocation/qmldir DESTINATION ${MODULE_PATH}/GeoLocation/)
148
149install(TARGETS clockutility DESTINATION ${MODULE_PATH}/Clock/Utility/)
150install(FILES modules/Clock/Utility/qmldir DESTINATION ${MODULE_PATH}/Clock/Utility/)
15185
152install(TARGETS stopwatch DESTINATION ${MODULE_PATH}/Stopwatch/)86install(TARGETS stopwatch DESTINATION ${MODULE_PATH}/Stopwatch/)
153install(FILES modules/Stopwatch/qmldir DESTINATION ${MODULE_PATH}/Stopwatch/)87install(FILES modules/Stopwatch/qmldir DESTINATION ${MODULE_PATH}/Stopwatch/)
15488
=== removed directory 'backend/modules/Alarm/Settings'
=== renamed file 'backend/modules/Alarm/Settings/backend.cpp' => 'backend/modules/Alarm/backend.cpp'
--- backend/modules/Alarm/Settings/backend.cpp 2015-08-14 05:34:49 +0000
+++ backend/modules/Alarm/backend.cpp 2015-08-26 23:37:55 +0000
@@ -19,13 +19,15 @@
19#include <QtQml>19#include <QtQml>
20#include <QtQml/QQmlContext>20#include <QtQml/QQmlContext>
21#include "backend.h"21#include "backend.h"
22#include "alarmsettings.h"22#include "settings.h"
23#include "sound.h"
2324
24void BackendPlugin::registerTypes(const char *uri)25void BackendPlugin::registerTypes(const char *uri)
25{26{
26 Q_ASSERT(uri == QLatin1String("Alarm.Settings"));27 Q_ASSERT(uri == QLatin1String("Alarm"));
2728
28 qmlRegisterType<AlarmSettings>(uri, 1, 0, "AlarmSettings");29 qmlRegisterType<Settings>(uri, 1, 0, "AlarmSettings");
30 qmlRegisterType<Sound>(uri, 1, 0, "AlarmSound");
29}31}
3032
31void BackendPlugin::initializeEngine(QQmlEngine *engine, const char *uri)33void BackendPlugin::initializeEngine(QQmlEngine *engine, const char *uri)
3234
=== renamed file 'backend/modules/Alarm/Settings/backend.h' => 'backend/modules/Alarm/backend.h'
=== renamed file 'backend/modules/Alarm/Settings/qmldir' => 'backend/modules/Alarm/qmldir'
--- backend/modules/Alarm/Settings/qmldir 2014-07-17 10:38:36 +0000
+++ backend/modules/Alarm/qmldir 2015-08-26 23:37:55 +0000
@@ -1,2 +1,2 @@
1module Alarm.Settings1module Alarm
2plugin alarmsettings2plugin alarm
33
=== renamed file 'backend/modules/Alarm/Settings/alarmsettings.cpp' => 'backend/modules/Alarm/settings.cpp'
--- backend/modules/Alarm/Settings/alarmsettings.cpp 2015-08-14 05:34:49 +0000
+++ backend/modules/Alarm/settings.cpp 2015-08-26 23:37:55 +0000
@@ -19,9 +19,9 @@
19#include <QDebug>19#include <QDebug>
20#include <QtDBus>20#include <QtDBus>
2121
22#include "alarmsettings.h"22#include "settings.h"
2323
24AlarmSettings::AlarmSettings(QObject *parent):24Settings::Settings(QObject *parent):
25 QObject(parent)25 QObject(parent)
26{26{
27 // On startup, retrieve all the settings values from Dbus27 // On startup, retrieve all the settings values from Dbus
@@ -37,7 +37,7 @@
37 SLOT(onSettingsChanged(QString, QVariantMap, QStringList)));37 SLOT(onSettingsChanged(QString, QVariantMap, QStringList)));
38}38}
3939
40void AlarmSettings::onSettingsChanged(const QString &interface,40void Settings::onSettingsChanged(const QString &interface,
41 const QVariantMap &properties,41 const QVariantMap &properties,
42 const QStringList & /*valid*/)42 const QStringList & /*valid*/)
43{43{
@@ -87,7 +87,7 @@
87 }87 }
88}88}
8989
90void AlarmSettings::refreshProperties()90void Settings::refreshProperties()
91{91{
92 QDBusInterface handlerPropertiesInterface92 QDBusInterface handlerPropertiesInterface
93 ("com.canonical.indicator.datetime",93 ("com.canonical.indicator.datetime",
@@ -111,7 +111,7 @@
111 m_vibration = map["HapticFeedback"].toString();111 m_vibration = map["HapticFeedback"].toString();
112}112}
113113
114void AlarmSettings::setDBusProperty(const QString &name, const QVariant &value)114void Settings::setDBusProperty(const QString &name, const QVariant &value)
115{115{
116 QDBusInterface handlerPropertiesInterface116 QDBusInterface handlerPropertiesInterface
117 ("com.canonical.indicator.datetime",117 ("com.canonical.indicator.datetime",
@@ -125,27 +125,27 @@
125 QVariant::fromValue(QDBusVariant(value)));125 QVariant::fromValue(QDBusVariant(value)));
126}126}
127127
128int AlarmSettings::volume() const128int Settings::volume() const
129{129{
130 return m_volume;130 return m_volume;
131}131}
132132
133int AlarmSettings::duration() const133int Settings::duration() const
134{134{
135 return m_duration;135 return m_duration;
136}136}
137137
138int AlarmSettings::snoozeDuration() const138int Settings::snoozeDuration() const
139{139{
140 return m_snoozeDuration;140 return m_snoozeDuration;
141}141}
142142
143QString AlarmSettings::vibration() const143QString Settings::vibration() const
144{144{
145 return m_vibration;145 return m_vibration;
146}146}
147147
148void AlarmSettings::setVolume(int volume)148void Settings::setVolume(int volume)
149{149{
150 if(m_volume == volume) {150 if(m_volume == volume) {
151 // Don't send the volume over dbus if it is the same one already151 // Don't send the volume over dbus if it is the same one already
@@ -159,7 +159,7 @@
159 setDBusProperty("DefaultVolume", QVariant(m_volume));159 setDBusProperty("DefaultVolume", QVariant(m_volume));
160}160}
161161
162void AlarmSettings::setDuration(int duration)162void Settings::setDuration(int duration)
163{163{
164 if(m_duration == duration) {164 if(m_duration == duration) {
165 // Don't send the duration over dbus if it is the same one already165 // Don't send the duration over dbus if it is the same one already
@@ -173,7 +173,7 @@
173 setDBusProperty("Duration", QVariant(m_duration));173 setDBusProperty("Duration", QVariant(m_duration));
174}174}
175175
176void AlarmSettings::setSnoozeDuration(int snoozeDuration)176void Settings::setSnoozeDuration(int snoozeDuration)
177{177{
178 if(m_snoozeDuration == snoozeDuration) {178 if(m_snoozeDuration == snoozeDuration) {
179 // Don't send the snooze duration over dbus if it is the same one already179 // Don't send the snooze duration over dbus if it is the same one already
@@ -187,7 +187,7 @@
187 setDBusProperty("SnoozeDuration", QVariant(m_snoozeDuration));187 setDBusProperty("SnoozeDuration", QVariant(m_snoozeDuration));
188}188}
189189
190void AlarmSettings::setVibration(QString vibration)190void Settings::setVibration(QString vibration)
191{191{
192 if(m_vibration == vibration) {192 if(m_vibration == vibration) {
193 // Don't send the vibration mode over dbus if it is the same one already193 // Don't send the vibration mode over dbus if it is the same one already
194194
=== renamed file 'backend/modules/Alarm/Settings/alarmsettings.h' => 'backend/modules/Alarm/settings.h'
--- backend/modules/Alarm/Settings/alarmsettings.h 2015-08-14 05:34:49 +0000
+++ backend/modules/Alarm/settings.h 2015-08-26 23:37:55 +0000
@@ -16,13 +16,13 @@
16 * along with this program. If not, see <http://www.gnu.org/licenses/>.16 * along with this program. If not, see <http://www.gnu.org/licenses/>.
17 */17 */
1818
19#ifndef ALARMSETTINGS_H19#ifndef SETTINGS_H
20#define ALARMSETTINGS_H20#define SETTINGS_H
2121
22#include <QObject>22#include <QObject>
23#include <QVariant>23#include <QVariant>
2424
25class AlarmSettings: public QObject25class Settings: public QObject
26{26{
27 Q_OBJECT27 Q_OBJECT
2828
@@ -51,7 +51,7 @@
51 NOTIFY vibrationChanged)51 NOTIFY vibrationChanged)
5252
53public:53public:
54 AlarmSettings(QObject *parent = 0);54 Settings(QObject *parent = 0);
5555
56 // Function to return the alarm volume56 // Function to return the alarm volume
57 int volume() const;57 int volume() const;
5858
=== renamed file 'backend/modules/Clock/Utility/alarmsound.cpp' => 'backend/modules/Alarm/sound.cpp'
--- backend/modules/Clock/Utility/alarmsound.cpp 2015-08-24 21:15:05 +0000
+++ backend/modules/Alarm/sound.cpp 2015-08-26 23:37:55 +0000
@@ -20,21 +20,21 @@
20#include <QFileInfo>20#include <QFileInfo>
21#include <QStandardPaths>21#include <QStandardPaths>
2222
23#include "alarmsound.h"23#include "sound.h"
2424
25AlarmSound::AlarmSound(QObject *parent):25Sound::Sound(QObject *parent):
26 QObject(parent),26 QObject(parent),
27 m_customAlarmDir(QStandardPaths::standardLocations(QStandardPaths::AppDataLocation).first() + "/CustomSounds/"),27 m_customAlarmDir(QStandardPaths::standardLocations(QStandardPaths::AppDataLocation).first() + "/CustomSounds/"),
28 m_defaultAlarmDir("/usr/share/sounds/ubuntu/ringtones/")28 m_defaultAlarmDir("/usr/share/sounds/ubuntu/ringtones/")
29{29{
30}30}
3131
32QString AlarmSound::customAlarmSoundDirectory() const32QString Sound::customAlarmSoundDirectory() const
33{33{
34 return m_customAlarmDir;34 return m_customAlarmDir;
35}35}
3636
37void AlarmSound::deleteCustomAlarmSound(const QString &soundName)37void Sound::deleteCustomAlarmSound(const QString &soundName)
38{38{
39 QDir dir(m_customAlarmDir);39 QDir dir(m_customAlarmDir);
40 if (dir.exists(soundName))40 if (dir.exists(soundName))
@@ -43,14 +43,14 @@
43 }43 }
44}44}
4545
46void AlarmSound::prepareToAddAlarmSound(const QString &soundPath)46void Sound::prepareToAddAlarmSound(const QString &soundPath)
47{47{
48 QFileInfo soundFile(soundPath);48 QFileInfo soundFile(soundPath);
49 QString soundFileName = soundFile.fileName();49 QString soundFileName = soundFile.fileName();
50 deleteCustomAlarmSound(soundFileName);50 deleteCustomAlarmSound(soundFileName);
51}51}
5252
53void AlarmSound::createCustomAlarmSoundDirectory()53void Sound::createCustomAlarmSoundDirectory()
54{54{
55 QDir dir(m_customAlarmDir);55 QDir dir(m_customAlarmDir);
5656
@@ -62,7 +62,7 @@
62 dir.mkpath(m_customAlarmDir);62 dir.mkpath(m_customAlarmDir);
63}63}
6464
65bool AlarmSound::isAlarmSoundValid(const QString &soundFileName)65bool Sound::isAlarmSoundValid(const QString &soundFileName)
66{66{
67 QFileInfo soundFile;67 QFileInfo soundFile;
6868
@@ -82,18 +82,18 @@
82 }82 }
83}83}
8484
85bool AlarmSound::isAlarmSoundValid(const QUrl &soundUrl)85bool Sound::isAlarmSoundValid(const QUrl &soundUrl)
86{86{
87 QFileInfo soundFile;87 QFileInfo soundFile;
88 return soundFile.exists(soundUrl.path());88 return soundFile.exists(soundUrl.path());
89}89}
9090
91QString AlarmSound::getDefaultAlarmSoundPath(const QString &soundFileName) const91QString Sound::getDefaultAlarmSoundPath(const QString &soundFileName) const
92{92{
93 return m_defaultAlarmDir + soundFileName;93 return m_defaultAlarmDir + soundFileName;
94}94}
9595
96QString AlarmSound::getSoundName(const QString &soundPath) const96QString Sound::getSoundName(const QString &soundPath) const
97{97{
98 QFileInfo soundFile(soundPath);98 QFileInfo soundFile(soundPath);
99 return soundFile.baseName();99 return soundFile.baseName();
100100
=== renamed file 'backend/modules/Clock/Utility/alarmsound.h' => 'backend/modules/Alarm/sound.h'
--- backend/modules/Clock/Utility/alarmsound.h 2015-08-24 16:32:44 +0000
+++ backend/modules/Alarm/sound.h 2015-08-26 23:37:55 +0000
@@ -16,13 +16,13 @@
16 * along with this program. If not, see <http://www.gnu.org/licenses/>.16 * along with this program. If not, see <http://www.gnu.org/licenses/>.
17 */17 */
1818
19#ifndef ALARMSOUND_H19#ifndef SOUND_H
20#define ALARMSOUND_H20#define SOUND_H
2121
22#include <QObject>22#include <QObject>
23#include <QUrl>23#include <QUrl>
2424
25class AlarmSound: public QObject25class Sound: public QObject
26{26{
27 Q_OBJECT27 Q_OBJECT
2828
@@ -32,7 +32,7 @@
32 CONSTANT)32 CONSTANT)
3333
34public:34public:
35 AlarmSound(QObject *parent = 0);35 Sound(QObject *parent = 0);
3636
37 QString customAlarmSoundDirectory() const;37 QString customAlarmSoundDirectory() const;
3838
3939
=== removed directory 'backend/modules/Clock/Utility'
=== removed directory 'backend/modules/DateTime'
=== removed file 'backend/modules/DateTime/backend.cpp'
--- backend/modules/DateTime/backend.cpp 2015-08-14 05:34:49 +0000
+++ backend/modules/DateTime/backend.cpp 1970-01-01 00:00:00 +0000
@@ -1,34 +0,0 @@
1/*
2 * Copyright (C) 2014-2015 Canonical Ltd
3 *
4 * This file is part of Ubuntu Clock App
5 *
6 * Ubuntu Clock App is free software: you can redistribute it and/or modify
7 * it under the terms of the GNU General Public License version 3 as
8 * published by the Free Software Foundation.
9 *
10 * Ubuntu Clock App is distributed in the hope that it will be useful,
11 * but WITHOUT ANY WARRANTY; without even the implied warranty of
12 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
13 * GNU General Public License for more details.
14 *
15 * You should have received a copy of the GNU General Public License
16 * along with this program. If not, see <http://www.gnu.org/licenses/>.
17 */
18
19#include <QtQml>
20#include <QtQml/QQmlContext>
21#include "backend.h"
22#include "datetime.h"
23
24void BackendPlugin::registerTypes(const char *uri)
25{
26 Q_ASSERT(uri == QLatin1String("DateTime"));
27
28 qmlRegisterType<DateTime>(uri, 1, 0, "DateTime");
29}
30
31void BackendPlugin::initializeEngine(QQmlEngine *engine, const char *uri)
32{
33 QQmlExtensionPlugin::initializeEngine(engine, uri);
34}
350
=== removed file 'backend/modules/DateTime/backend.h'
--- backend/modules/DateTime/backend.h 2015-08-14 05:34:49 +0000
+++ backend/modules/DateTime/backend.h 1970-01-01 00:00:00 +0000
@@ -1,35 +0,0 @@
1/*
2 * Copyright (C) 2014-2015 Canonical Ltd
3 *
4 * This file is part of Ubuntu Clock App
5 *
6 * Ubuntu Clock App is free software: you can redistribute it and/or modify
7 * it under the terms of the GNU General Public License version 3 as
8 * published by the Free Software Foundation.
9 *
10 * Ubuntu Clock App is distributed in the hope that it will be useful,
11 * but WITHOUT ANY WARRANTY; without even the implied warranty of
12 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
13 * GNU General Public License for more details.
14 *
15 * You should have received a copy of the GNU General Public License
16 * along with this program. If not, see <http://www.gnu.org/licenses/>.
17 */
18
19#ifndef BACKEND_PLUGIN_H
20#define BACKEND_PLUGIN_H
21
22#include <QtQml/QQmlEngine>
23#include <QtQml/QQmlExtensionPlugin>
24
25class BackendPlugin : public QQmlExtensionPlugin
26{
27 Q_OBJECT
28 Q_PLUGIN_METADATA(IID "org.qt-project.Qt.QQmlExtensionInterface")
29
30public:
31 void registerTypes(const char *uri);
32 void initializeEngine(QQmlEngine *engine, const char *uri);
33};
34#endif // BACKEND_PLUGIN_H
35
360
=== removed file 'backend/modules/DateTime/qmldir'
--- backend/modules/DateTime/qmldir 2014-07-26 09:34:15 +0000
+++ backend/modules/DateTime/qmldir 1970-01-01 00:00:00 +0000
@@ -1,2 +0,0 @@
1module DateTime
2plugin datetime
30
=== removed directory 'backend/modules/GeoLocation'
=== removed file 'backend/modules/GeoLocation/backend.cpp'
--- backend/modules/GeoLocation/backend.cpp 2015-08-14 05:34:49 +0000
+++ backend/modules/GeoLocation/backend.cpp 1970-01-01 00:00:00 +0000
@@ -1,34 +0,0 @@
1/*
2 * Copyright (C) 2014-2015 Canonical Ltd
3 *
4 * This file is part of Ubuntu Clock App
5 *
6 * Ubuntu Clock App is free software: you can redistribute it and/or modify
7 * it under the terms of the GNU General Public License version 3 as
8 * published by the Free Software Foundation.
9 *
10 * Ubuntu Clock App is distributed in the hope that it will be useful,
11 * but WITHOUT ANY WARRANTY; without even the implied warranty of
12 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
13 * GNU General Public License for more details.
14 *
15 * You should have received a copy of the GNU General Public License
16 * along with this program. If not, see <http://www.gnu.org/licenses/>.
17 */
18
19#include <QtQml>
20#include <QtQml/QQmlContext>
21#include "backend.h"
22#include "geolocation.h"
23
24void BackendPlugin::registerTypes(const char *uri)
25{
26 Q_ASSERT(uri == QLatin1String("GeoLocation"));
27
28 qmlRegisterType<GeoLocation>(uri, 1, 0, "GeoLocation");
29}
30
31void BackendPlugin::initializeEngine(QQmlEngine *engine, const char *uri)
32{
33 QQmlExtensionPlugin::initializeEngine(engine, uri);
34}
350
=== removed file 'backend/modules/GeoLocation/backend.h'
--- backend/modules/GeoLocation/backend.h 2015-08-14 05:34:49 +0000
+++ backend/modules/GeoLocation/backend.h 1970-01-01 00:00:00 +0000
@@ -1,35 +0,0 @@
1/*
2 * Copyright (C) 2014-2015 Canonical Ltd
3 *
4 * This file is part of Ubuntu Clock App
5 *
6 * Ubuntu Clock App is free software: you can redistribute it and/or modify
7 * it under the terms of the GNU General Public License version 3 as
8 * published by the Free Software Foundation.
9 *
10 * Ubuntu Clock App is distributed in the hope that it will be useful,
11 * but WITHOUT ANY WARRANTY; without even the implied warranty of
12 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
13 * GNU General Public License for more details.
14 *
15 * You should have received a copy of the GNU General Public License
16 * along with this program. If not, see <http://www.gnu.org/licenses/>.
17 */
18
19#ifndef BACKEND_PLUGIN_H
20#define BACKEND_PLUGIN_H
21
22#include <QtQml/QQmlEngine>
23#include <QtQml/QQmlExtensionPlugin>
24
25class BackendPlugin : public QQmlExtensionPlugin
26{
27 Q_OBJECT
28 Q_PLUGIN_METADATA(IID "org.qt-project.Qt.QQmlExtensionInterface")
29
30public:
31 void registerTypes(const char *uri);
32 void initializeEngine(QQmlEngine *engine, const char *uri);
33};
34#endif // BACKEND_PLUGIN_H
35
360
=== removed file 'backend/modules/GeoLocation/qmldir'
--- backend/modules/GeoLocation/qmldir 2014-11-04 15:58:05 +0000
+++ backend/modules/GeoLocation/qmldir 1970-01-01 00:00:00 +0000
@@ -1,2 +0,0 @@
1module GeoLocation
2plugin geolocation
30
=== removed directory 'backend/modules/Timezone'
=== removed file 'backend/modules/Timezone/backend.cpp'
--- backend/modules/Timezone/backend.cpp 2015-08-14 05:34:49 +0000
+++ backend/modules/Timezone/backend.cpp 1970-01-01 00:00:00 +0000
@@ -1,39 +0,0 @@
1/*
2 * Copyright (C) 2014-2015 Canonical Ltd
3 *
4 * This file is part of Ubuntu Clock App
5 *
6 * Ubuntu Clock App is free software: you can redistribute it and/or modify
7 * it under the terms of the GNU General Public License version 3 as
8 * published by the Free Software Foundation.
9 *
10 * Ubuntu Clock App is distributed in the hope that it will be useful,
11 * but WITHOUT ANY WARRANTY; without even the implied warranty of
12 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
13 * GNU General Public License for more details.
14 *
15 * You should have received a copy of the GNU General Public License
16 * along with this program. If not, see <http://www.gnu.org/licenses/>.
17 */
18
19#include <QtQml>
20#include <QtQml/QQmlContext>
21#include "backend.h"
22#include "timezonemodel.h"
23#include "generictimezonemodel.h"
24#include "jsontimezonemodel.h"
25#include "statictimezonemodel.h"
26
27void BackendPlugin::registerTypes(const char *uri)
28{
29 Q_ASSERT(uri == QLatin1String("Timezone"));
30
31 qmlRegisterType<GenericTimeZoneModel>(uri, 1, 0, "GenericTimeZoneModel");
32 qmlRegisterType<JsonTimeZoneModel>(uri, 1, 0, "JsonTimeZoneModel");
33 qmlRegisterType<StaticTimeZoneModel>(uri, 1, 0, "StaticTimeZoneModel");
34}
35
36void BackendPlugin::initializeEngine(QQmlEngine *engine, const char *uri)
37{
38 QQmlExtensionPlugin::initializeEngine(engine, uri);
39}
400
=== removed file 'backend/modules/Timezone/backend.h'
--- backend/modules/Timezone/backend.h 2015-08-14 05:34:49 +0000
+++ backend/modules/Timezone/backend.h 1970-01-01 00:00:00 +0000
@@ -1,35 +0,0 @@
1/*
2 * Copyright (C) 2014-2015 Canonical Ltd
3 *
4 * This file is part of Ubuntu Clock App
5 *
6 * Ubuntu Clock App is free software: you can redistribute it and/or modify
7 * it under the terms of the GNU General Public License version 3 as
8 * published by the Free Software Foundation.
9 *
10 * Ubuntu Clock App is distributed in the hope that it will be useful,
11 * but WITHOUT ANY WARRANTY; without even the implied warranty of
12 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
13 * GNU General Public License for more details.
14 *
15 * You should have received a copy of the GNU General Public License
16 * along with this program. If not, see <http://www.gnu.org/licenses/>.
17 */
18
19#ifndef BACKEND_PLUGIN_H
20#define BACKEND_PLUGIN_H
21
22#include <QtQml/QQmlEngine>
23#include <QtQml/QQmlExtensionPlugin>
24
25class BackendPlugin : public QQmlExtensionPlugin
26{
27 Q_OBJECT
28 Q_PLUGIN_METADATA(IID "org.qt-project.Qt.QQmlExtensionInterface")
29
30public:
31 void registerTypes(const char *uri);
32 void initializeEngine(QQmlEngine *engine, const char *uri);
33};
34#endif // BACKEND_PLUGIN_H
35
360
=== removed file 'backend/modules/Timezone/qmldir'
--- backend/modules/Timezone/qmldir 2014-07-17 10:38:36 +0000
+++ backend/modules/Timezone/qmldir 1970-01-01 00:00:00 +0000
@@ -1,2 +0,0 @@
1module Timezone
2plugin timezone
30
=== renamed directory 'backend/modules/Clock' => 'backend/modules/WorldClock'
=== renamed file 'backend/modules/Clock/Utility/backend.cpp' => 'backend/modules/WorldClock/backend.cpp'
--- backend/modules/Clock/Utility/backend.cpp 2015-08-24 15:08:21 +0000
+++ backend/modules/WorldClock/backend.cpp 2015-08-26 23:37:55 +0000
@@ -19,13 +19,22 @@
19#include <QtQml>19#include <QtQml>
20#include <QtQml/QQmlContext>20#include <QtQml/QQmlContext>
21#include "backend.h"21#include "backend.h"
22#include "alarmsound.h"22#include "geolocation.h"
23#include "datetime.h"
24#include "timezonemodel.h"
25#include "generictimezonemodel.h"
26#include "jsontimezonemodel.h"
27#include "statictimezonemodel.h"
2328
24void BackendPlugin::registerTypes(const char *uri)29void BackendPlugin::registerTypes(const char *uri)
25{30{
26 Q_ASSERT(uri == QLatin1String("Clock.Utility"));31 Q_ASSERT(uri == QLatin1String("WorldClock"));
2732
28 qmlRegisterType<AlarmSound>(uri, 1, 0, "AlarmSound");33 qmlRegisterType<GeoLocation>(uri, 1, 0, "GeoLocation");
34 qmlRegisterType<DateTime>(uri, 1, 0, "DateTime");
35 qmlRegisterType<GenericTimeZoneModel>(uri, 1, 0, "GenericTimeZoneModel");
36 qmlRegisterType<JsonTimeZoneModel>(uri, 1, 0, "JsonTimeZoneModel");
37 qmlRegisterType<StaticTimeZoneModel>(uri, 1, 0, "StaticTimeZoneModel");
29}38}
3039
31void BackendPlugin::initializeEngine(QQmlEngine *engine, const char *uri)40void BackendPlugin::initializeEngine(QQmlEngine *engine, const char *uri)
3241
=== renamed file 'backend/modules/Clock/Utility/backend.h' => 'backend/modules/WorldClock/backend.h'
=== renamed file 'backend/modules/DateTime/datetime.cpp' => 'backend/modules/WorldClock/datetime.cpp'
=== renamed file 'backend/modules/DateTime/datetime.h' => 'backend/modules/WorldClock/datetime.h'
=== renamed file 'backend/modules/Timezone/generictimezonemodel.cpp' => 'backend/modules/WorldClock/generictimezonemodel.cpp'
=== renamed file 'backend/modules/Timezone/generictimezonemodel.h' => 'backend/modules/WorldClock/generictimezonemodel.h'
=== renamed file 'backend/modules/GeoLocation/geolocation.cpp' => 'backend/modules/WorldClock/geolocation.cpp'
=== renamed file 'backend/modules/GeoLocation/geolocation.h' => 'backend/modules/WorldClock/geolocation.h'
=== renamed file 'backend/modules/Timezone/jsontimezonemodel.cpp' => 'backend/modules/WorldClock/jsontimezonemodel.cpp'
=== renamed file 'backend/modules/Timezone/jsontimezonemodel.h' => 'backend/modules/WorldClock/jsontimezonemodel.h'
=== renamed file 'backend/modules/Clock/Utility/qmldir' => 'backend/modules/WorldClock/qmldir'
--- backend/modules/Clock/Utility/qmldir 2015-08-19 13:29:52 +0000
+++ backend/modules/WorldClock/qmldir 2015-08-26 23:37:55 +0000
@@ -1,2 +1,2 @@
1module Clock.Utility1module WorldClock
2plugin clockutility2plugin worldclock
33
=== renamed file 'backend/modules/Timezone/statictimezonemodel.cpp' => 'backend/modules/WorldClock/statictimezonemodel.cpp'
=== renamed file 'backend/modules/Timezone/statictimezonemodel.h' => 'backend/modules/WorldClock/statictimezonemodel.h'
=== renamed file 'backend/modules/Timezone/timezonemodel.cpp' => 'backend/modules/WorldClock/timezonemodel.cpp'
=== renamed file 'backend/modules/Timezone/timezonemodel.h' => 'backend/modules/WorldClock/timezonemodel.h'
=== modified file 'debian/changelog'
--- debian/changelog 2015-08-25 11:36:52 +0000
+++ debian/changelog 2015-08-26 23:37:55 +0000
@@ -29,6 +29,7 @@
29 * Fixed edited alarm not being saveable if at any point of editing the alarm29 * Fixed edited alarm not being saveable if at any point of editing the alarm
30 clock app loses focus (LP: #1487789)30 clock app loses focus (LP: #1487789)
31 * Fixed edited alarms when saved not updating the alarms listview (LP: #1488439)31 * Fixed edited alarms when saved not updating the alarms listview (LP: #1488439)
32 * Reorganized backend c++ plugins with proper naming and folder structures.
3233
33 [Victor Thompson]34 [Victor Thompson]
34 * Show all README files in QtCreator 35 * Show all README files in QtCreator
3536
=== modified file 'tests/unit/MockClockApp.qml'
--- tests/unit/MockClockApp.qml 2015-08-17 12:27:20 +0000
+++ tests/unit/MockClockApp.qml 2015-08-26 23:37:55 +0000
@@ -17,7 +17,7 @@
17 */17 */
1818
19import QtQuick 2.419import QtQuick 2.4
20import DateTime 1.020import WorldClock 1.0
21import U1db 1.0 as U1db21import U1db 1.0 as U1db
22import Ubuntu.Components 1.222import Ubuntu.Components 1.2
23import "../../app"23import "../../app"
2424
=== modified file 'tests/unit/tst_alarm.qml'
--- tests/unit/tst_alarm.qml 2015-08-12 16:57:29 +0000
+++ tests/unit/tst_alarm.qml 2015-08-26 23:37:55 +0000
@@ -17,7 +17,7 @@
17 */17 */
1818
19import QtQuick 2.419import QtQuick 2.4
20import DateTime 1.020import WorldClock 1.0
21import Ubuntu.Components 1.221import Ubuntu.Components 1.2
22import "../../app/alarm"22import "../../app/alarm"
2323

Subscribers

People subscribed via source and target branches