[Kde-pim] Review Request 119096: Used the model in the composer.

Abhijeet Nikam connect08nikam at gmail.com
Thu Jul 3 10:23:07 BST 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119096/
-----------------------------------------------------------

(Updated July 3, 2014, 9:23 a.m.)


Review request for KDEPIM, Kevin Krammer and Michael Bohlender.


Changes
-------

Corrected the coding style for the constructor.


Repository: kdepim


Description
-------

Added a slot addRecipients() to the composer. Used the model for input of values. Should I change name from model to receiverModel in Q_PROPERTY? Please ignore the receivermodel.cpp file. I have just corrected the coding style and added setRoleNames() , which was missing due to an incorrect commit.


Diffs (updated)
-----

  mobile/api/mail/composer.h fb6c7ef 
  mobile/api/mail/composer.cpp dc9cafe 
  mobile/api/mail/receivermodel.cpp 5197e18 

Diff: https://git.reviewboard.kde.org/r/119096/diff/


Testing
-------

Done. Sent a mail using the values from model. Checked storage in model using qdebug().Used the following code for testing.

import org.kde.pim.mail 0.1 as Mail

Rectangle {
  id: app
  
    Mail.Composer {
	id: composer
	from: "connect08nikam at gmail.com"
	subject: "test mail"
	body: "testing"
    }

    ListView {
	model: composer.model 
        width: 180
        height: 100
	delegate: 
	    Text { text: model.email + ":" + model.type }
  
  }

    MouseArea {
	anchors.fill : parent  
	onClicked: {
	    composer.addRecipient ( "connect08nikam at gmail.com" , 1 )
	    composer.send()
	}
    }

  
}


Thanks,

Abhijeet Nikam

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/



More information about the kde-pim mailing list