From fbec51f476a66d2104e57f2dafa8eec082166b46 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Thibault=20Delavall=C3=A9e?= Date: Wed, 30 May 2018 14:28:00 +0200 Subject: [PATCH] [IMP] mail: improve default channel picture Purpose is to avoid having void image in systray when there are notifications linked to channels that have no picture. Two small improvements are done on channel model * change default picture for channel to be more inline with Odoo design guidelines; * override create to ensure there is a default image when creating the channel through the discuss interface; This commit is linked to task ID 48228. --- addons/mail/models/mail_channel.py | 5 +++++ addons/mail/static/src/img/groupdefault.png | Bin 3092 -> 2461 bytes 2 files changed, 5 insertions(+) diff --git a/addons/mail/models/mail_channel.py b/addons/mail/models/mail_channel.py index 8d8fc13a407..9da0862376f 100644 --- a/addons/mail/models/mail_channel.py +++ b/addons/mail/models/mail_channel.py @@ -115,6 +115,11 @@ class Channel(models.Model): @api.model def create(self, vals): + # ensure image at quick create + if not vals.get('image'): + defaults = self.default_get(['image']) + vals['image'] = defaults['image'] + tools.image_resize_images(vals) # Create channel and alias channel = super(Channel, self.with_context( diff --git a/addons/mail/static/src/img/groupdefault.png b/addons/mail/static/src/img/groupdefault.png index cf160022470e0f0d1c8928b9364bf99349bbee45..5d628e992e7cb65eca6ac0da2e062f67366bfbda 100644 GIT binary patch literal 2461 zcmaJ@XH-+!7EU6%P(~4~z~ICHZx9VB1SBCSCDai?$c%^%CL{p@rjSGlAWD#`1f)q5 zL_n%^btnP>q)A6bN)S+rKBII2rM$!u=lyuTyVkwu?6daw?QfT})(y2eX|{cv;x+&P zu-(EOXDeR)B(L-)@ppbCa#_46a7^qub_{2ZCy_+~7?K%I6tD%2=t8lj5XoL1Efg#O zu<2h^ygkR>>I8PHS7i zt(e9XSE{);i(=<}5>N7WC!xuZuXe8bdTM%VX);(YFa#QnhQX0ABoZq2fU>>l93mG= zXRB;5;3#Yoi^}9s8Fa9Ok?6#@$kB(0mHwRqjrpCH&i+;=alv3*A`^ztgiBJ|aI~`e z|E@IJcW*Ywmhz9@|4Gcodod|6TMCxs~L1%M_bP~k^rwl9lKZP!yc$Rpyqg5@V z005xK<^n zZh`N_dpkT)omsZ>`Abn$c0*xJv|Gr+%>4VYcgu@Q@gCthp{Z-DtCK@x!##s3-Z3%m zp%cP4*Uw*_{W#N8+a7ZEa>zN~rsCRxw%)b1wKTs3VMkwvf6~&zVqRE!gmXYn=-t-8 z8cXBuKgqBDF!BCP|43u8;OV1=WN!4cx{l#z!p}>~y^Y=19j{~s-+4PSk#sTg->=^` zm)6BG!rt_aM3Aq&>KQCcDSntx@NrT!{9HIAn(BM{Vs2_SgO^l(r{sBkXYP%(M@fY< zqUp)u@q(L~Q|~_9z7!SD3XgORn*A^{)<1gNGb-(BLOL&TasE?gVDhv2&e7gi)9+$D?G<+R*JPFzMrGf<5`T*kmLHzc+tkxj*HN2WSsasd$2<0~ zq{4TjlU>!VF&9D;*b(96Ytil@t!0h?vBcq;Io(8E1DBIH@gMxgB^Y0B>$T1@|K5BhPwI(+h68h zznkutSd&#AM&vCnEDEwK3U6l4eViR1d|j3Cs3bNwiW>ZBcA>4TDQ6{r005BMV}Uco za|c98E_QZ?yMycyd{w^(J1vc1{vqaZ&UY>3@Ye~oAoY+wpQ)a8J(xx@$8`L_aKPkL%Xeg>HmTtLJqKa{paRowCH$U*(+XZqkkQm^D)9LWH_N5go zd+H&am@Iy0-F?m1)gu) zna9ob@BQZRpc59ae%-B5wWT3VgSXJEKHeYrlK`Os%oI(H^)Dj@R1IEx^L|trw$ca( z@+tYr`+O!OB_lBkQN@@otIg@QRx8@BkhHw1I;0#sm@V}=@>FxFM<(=~nHro52h76b zTx25+T=y9rn`7?05aWi|DJX zK#_ChN@8hVOMDj#f}Sr9w0xpvd}8#sqC&jvkyi)bq<0(o8YK(pyPsULKAnRp8LR%I1uTbV0zCig5Pgd`RbQxu{;`6c2 zvDR(siJi(()!Pem<+xC2SDtCAqeCi=FL0y+hlxsBss{*e9Y60os*h832XJL1LTrd@oc)l(Bxd({JkcU&&tj!gPjV8vF^s_y}K+_y8<7< zC$kNU-yW6Q`W)_SrY)P$6x)giWknXB^0V?&!Glca(0MX&zU2r28E{rl$p^>Qi!^YL zs4We4aK7vYktcW$<3}(58S+^ay^nh%Z0qRBnl0QU%Ml>*!0dhXXCwSAzk}LhR;s6! z+f08>l_w0;sno4p+RH#)I4EDV5+FEqjjubA1a{y#t!zOCD0m!@>b2V%U!t^n)iL$( zbliZ6?zz$RIaadDlDkX-pugwa-Q2jSBgc1|FImpltgr7S0N`D;g&^(tI>`@{g~>@= Iff2#~KWA@S^Z)<= literal 3092 zcmX|Dc{mh$7yixICfTD9V=!b(Q<3hN7!27-in2$zlx^%;rfefiw(MC$3uE7zy2;kH zTE6KupeXqdAb}<7{>MW$Up*sjwi)IVr#KPSV1^HXlcjn% z4fDXMl^5<0M8*WV2b%+&+gf*~W`Bz2eDoJl=@Qq2NI^#Vl(kE+CpGouwHcaL^#bhL zyaJ}To~K+v6k>y<(=0}wOud20@Z$`KQ&1LunTMyoBw=-PKPG;i2nwXAs#ZQSQU5Vj zEjqnSSsj?#J?gD-c5xY4%fk=;hNmm?@ygJQ4YRNiQ;^0FSg?Bk3fb0SpTA#_R>HPg zFw?ZQ9Ta=Os8&yP-P`f%r%@=k!quse-se-p%C!JEHw!GROg;}>x`M_-nZ!q#mfsK-vJ$gl=M-{#yc!po?V;uq_v{*0zve4yIr?Z1i^E-8 zQCnl_H(usG-xkVc-w^jZJ2sIHWkGnpxPBZ{Cg6WXNEkFRQ4iU?TVbg;G$dk#q_WX+;*dtk1&_h^e{+I-D(Xw%x=hiVR z;IBKn71~R&Dg6VmDzjN%{WHd(SzPL|sCJ4BCI!yUU34zfCBkD0lxUEqE3cQ+((%H>)a&LMdV8HAcbbbHnO707hQ6#h9z9v<}t(pU^*CUJhy!C^Uh z0MFJuERH*owq6knB8g^Z6gb(w{%?ic5xAEJWv)UiKQ*n)E<}ik!zdb^g5m7MG`ufW zy(HHll+D?SJ~5wHLo35vV;FDeKjJ4gnt_MU6NOO|WvC*!fB`f)df64FqsOM`|S zCODrj6X8zUwqT0e!>ZUKYkXj zF?wcND+@}4-+xi^qK653!s2rUo1}e0uS6G^kasv68!htBtkXp@axH$y;Ej3>e1^|w zr5g<^eKp3k^{*^0L`9MNQ7J5tjTw$m9CEr3LHL>XYzXz9a^W^YBFp8k+6|u*PQrjb zJJHM0an$%YvvmON%m+rcLj-k*9OZaji33G~)6RV!tSu3isS_${*px)0c=gmn)hE;_ z5Ck8OkE56IfGs=ohs)$PwFt05?Nr3ER(nJ%Y`2L&sy(U-h`kKnzp4Xq*2XEw$SZ32QpeZXh$rL4Icdla|yy)3n$ zvuoZgAYqL1P1p_Xb1fwNpwAW`4BDD3*^|#~LyNXhMal}gYQimu8MiOrZ3QoT<0eOJ zf2H(oD3`IB*7)}ivy-XFnro-z{W%+BZU%}2l7_j)&|M9Ie$SkpB zbN9a8`I$C@uEE5b8#X>%vQK|<5qxcYKL7As%Mf2@F&Ki`W>Ul@PDlQrukMHfIJ#TM1U5DWR9x3pey>g6YMBhR3{y%Pzo!}_=1 z#g8fMs0DMznsW1S{-E^;emtlvu{e<8WFcS~ny0xU(ncmTq-q&L0~L}nR<>c@d;#RZ zl7>L3^xF}W8G-YyCpW{k>nBtx+#8sm*5okV5tla`CNSxz-fEB;%L^wEhimv^)bfkr z0En578aWiM&abCE7%_lGw^bzxGA%69XeZaE`CRWhncsSmzo}{>GFx@=a40;fn%C}} zf;+9Z7|)O0O>}uS6WnTR^L$4IM6c|wwhV7NVQhUTO9(w5v~_OXqVBtERh(hVIrA(l zz|R*>-tMm6^{>SO!lOLwXJ%kfM473B|R`&A?y3)7`e{Zf3(_`Z1NC}g!#wfHF5LSFU{ zEP|$|*>~`bGhrH@-J_QXzsC10C-JN0?WJq_U6xlx(tY~QUQkqocPr%9?rpl`Spndh zJ4_w84wsRsq(wz(N!HaVBvEtG04!_Eiu;V@8&@(b$*Qih;qH?_hTVP*~cg^k+w5oGW zE?vxmCYjU6*`9gMUiw*}bJw~5;n~0&feY<9dis5YX9m;K-9l*oAQ`2B^@H6t1EfOk z?Alm-s#<(yN=iy;X{QkLeE7=TT)Hl@#ttJL)JhteX+!eG_E`#N+g&mm5n585%(dW+ z)ukka9;oH@q7m;#KYymRc#wk6>9PJ{UsYS{mvB$(oTXOa{EkX)P zoev{FMcrvbodjrfqL`|GA1a>eHnVsk`E0}YOxIBtTTSBlc*%m7B}DgiJ45&S^%w_1 zy=kL|?L54kPejt_LFdn(QBCX&TC$O(Y|F{XN#8&XvVaI5n=@eRwFleG__*_(j&XDg zsnu=F?Rm7Zc?)TIWFo$~zTQvwX~`17`c+dCa#2T*BNsp2P=^1|EbzqH_1+p$=fH5p zaDXk3USZv=59w*xS^az}t~2G-qdSvr(b3xMSgz5rF{#M!OXwTNWj9vLo+X;KWa(}sAS4@&tO$4@-a(>BJDH0_`K4;9m}?EnA(