From a738115b0a81f46ceeee9e0da2f22ca7fe978fe2 Mon Sep 17 00:00:00 2001
From: cornernote <cornernote@gmail.com>
Date: Wed, 12 Aug 2015 09:42:36 +0930
Subject: [PATCH] added no image and tweak display fields

---
 src/views/mod/_view.php |  32 ++++++++++++++++++++++----------
 src/views/mod/view.php  |  10 +++++-----
 web/img/no-image.png    | Bin 0 -> 4589 bytes
 3 files changed, 27 insertions(+), 15 deletions(-)
 create mode 100644 web/img/no-image.png

diff --git a/src/views/mod/_view.php b/src/views/mod/_view.php
index d700f81..1ec73be 100644
--- a/src/views/mod/_view.php
+++ b/src/views/mod/_view.php
@@ -1,8 +1,7 @@
 <?php
 
-use app\components\Git;
+use kartik\detail\DetailView;
 use yii\helpers\Html;
-use yii\widgets\DetailView;
 
 /* @var $this yii\web\View */
 /* @var $model app\models\Package */
@@ -13,27 +12,40 @@ use yii\widgets\DetailView;
     <h2><?= Html::a(Html::encode($model->name), ['view', 'name' => $model->name]) ?></h2>
 
     <div class="row">
+        <div class="col-sm-2">
+            <?php
+            if ($model->screenshots) {
+                echo Html::a(Html::img($model->screenshots[0], ['class' => 'thumbnail']), ['view', 'name' => $model->name]);
+            } else {
+                echo Html::a(Html::img('@web/img/no-image.png', ['class' => 'thumbnail']), ['view', 'name' => $model->name]);
+            }
+            ?>
+        </div>
         <div class="col-sm-10">
             <?= DetailView::widget([
                 'model' => $model,
+                'hideIfEmpty' => true,
                 'attributes' => [
                     'description',
                     'keywords',
+                    [
+                        'label' => 'Authors',
+                        'value' => $model->getAuthorsHtml(),
+                        'format' => 'raw',
+                    ],
+                    [
+                        'label' => 'License',
+                        'value' => $model->getLicenseHtml(),
+                        'format' => 'raw',
+                    ],
                     [
                         'label' => 'Links',
-                        'value' => $model->getLinksHtml(true),
+                        'value' => $model->getLinksHtml(),
                         'format' => 'raw',
                     ],
                 ],
             ]) ?>
         </div>
-        <div class="col-sm-2">
-            <?php
-            if ($model->screenshots) {
-                echo Html::a(Html::img($model->screenshots[0], ['class' => 'thumbnail']), ['view', 'name' => $model->name]);
-            }
-            ?>
-        </div>
     </div>
 
 </div>
diff --git a/src/views/mod/view.php b/src/views/mod/view.php
index 7119c53..b08466d 100644
--- a/src/views/mod/view.php
+++ b/src/views/mod/view.php
@@ -37,11 +37,6 @@ $this->params['breadcrumbs'][] = $this->title;
                     'name',
                     //'description',
                     'keywords',
-                    [
-                        'label' => 'Links',
-                        'value' => $model->getLinksHtml(false),
-                        'format' => 'raw',
-                    ],
                     [
                         'label' => 'Authors',
                         'value' => $model->getAuthorsHtml(),
@@ -52,6 +47,11 @@ $this->params['breadcrumbs'][] = $this->title;
                         'value' => $model->getLicenseHtml(),
                         'format' => 'raw',
                     ],
+                    [
+                        'label' => 'Links',
+                        'value' => $model->getLinksHtml(),
+                        'format' => 'raw',
+                    ],
                     //'hits',
                     'created_at',
                     //[
diff --git a/web/img/no-image.png b/web/img/no-image.png
new file mode 100644
index 0000000000000000000000000000000000000000..49a5e2dc89d891ca7c475fade168d6d4a48e1bc0
GIT binary patch
literal 4589
zcmd^Dd03L^+Q;!s^;KF^IVNs|HCZl*jv^>niJDWQm58~cprHbaueoE5Q*$!2)XX+D
zO|x8Z8*>|_G+Z(<@mNvrn1&mgxZ%>l8as2Hb3EU5{{23#=el^`=e~cxdwG6v-S2zJ
z4re8|8N3+;0?An)#hwI#zCnRNGLf6Uk?vU6UyP7`DFGIaz$v;n5JF^-L5E3nPcp=s
zO7tP0Boj$tfz4!75J(nEv3CR<ZH}Y7=u}<es*i3cH9(36flSRp1BhOJWB}qx_My<w
zP*F7-3ZanDP=cY2o=t!``3&V~ID>pD9B1zp?&pOhLCr7_(@>O@fJz34kWi{WEeI8g
zhJNNnNv~J8VNl3t7r+k<{gRZUjUB|C&LBe!b@g<-^o-yTqaStQKk6ZWL~29y_27Ci
zxIRn|p##@L86r@6aL6AYs5Bab<c&IswfrNN^bQR@0{{Ui7%U_tL^s4hm(K8k!I4NL
zOiv%CudgHZ&<P5o0mM)pT9C#X1C|`*#h?TL6gmyE%1HF22LouRG}A97Py@cw(t`eI
zleA&5P+|ZKuB*3N(wd`<&3|^KQoni!0Vm1-$oJn82ib=OkYOjuLG)mTm-OJgHC96f
zpv)O$B0y)@)9L<eU9>wx2k1d(=m8M(Q-->52!TQ)(L;hh``g%{tZ6|2k>*9V#-gE8
z23-n;gwn@iEsq!-F~;g65OBB!{0P?A{4m@??=ZsH+#GIXv4+Lcy@IJ^8nA{X{R4|!
z7rQD2YJfB{mdv1>C6g=}bSmU?$tcSDxfrd>_Xn1=elCc0u`p>eu+@Y8(?PFoN!7D@
z`Bk>ko3Fwr)1+!=NM-$K*-;k+k|$bY58H>n9qaD(Q^Bi0>7VDh*uUANbLPMS!|cJj
zgW(CA#x3Z}cOo(r-LNkYCBJd{X$<VUZ?oK+LlNyqmP=%()wmPfyL|O#+H9Dv+wl^%
z_s6psqz{+NoLpI6Gz2;Nt2Zft7~e|QITx8P3qk;WSyIcI@xQl!0k4l=$No>j{{_E}
zeZAz@@ax$>SN?y$xZ>-xv$Hr`Tg5?+IJ_oeZek%D)G?p#8n8zPm-qcR=g$gwVz0~p
zR?9OaWbQ@UXihrQsYf+9GID9C^-hj$`h(uCo*p-M_uj59adZ~0dTL>Mc#1uPKo}XZ
zT&vH{jC9U#tKL-U`I{DjrM>urY3SrV$<>w=SZ_@PLed@}NP)Reh(xc;+0mbvqk=})
z%#(#~HQ<#K_(GqywC>8F{qc`W-(K6k&6xGGur)Kq+1dHM?NkeljnzX@nW?ubM9<os
z(RG5xtG%^RCQJ7TvQz5C+XO_biDn?rIcmBUu|g@7PdcxiJ|9!lD{7mC3uH9t8qUcW
zdYAW*EqbDo2lKkg6V?Xj)m4Xxo{JXedZT^Dh2j!*boj0^Wq$4$EQWP;E2AuPVEb7B
z_^rHrZoIr#SFo6wsl=Cw0!Obb_!up|hS0(`aGuA{pFZ!@h>+XXxs>%?VuB!}&ViGr
z{rgmz-|HoVter{c^E4&m$^QD00CsityV02&fm#E76{V#H%YtY9#%c$bxeW~sSCkPW
z$SPw=;|XJQyjAglQAgY68^gK2zWsZcp_<>bN2T2>?}0xKoDUP+*_U%0+J8DP*8nec
z@hnsJie?oSwgrp!5L~VWc3w4GakSa)s^rL%dn6y(@hKs#?BK!RVQ#9w%J!1O_^!OA
zUta#@Vq<)9&A!e@uZ*`g7U~f+)Qe`scvGg)KwPyyo$iBpr}9^Ih4?pM+*t9?$2YL<
z*4NidNDZxb5Gmo#&wVDr@)N=ggO$NHC;a6Y`t63JM;4yF&cA;5>2n<BigKP|Z;01D
zwW!BE=(85#cix<+FZjvI75a-y`vm8*;wN<%cc+Q|8F6dV1crrAZo@2UyT-qvESM8C
zwzO9j2Np`6;Uy=JF_O7|acF66Nk^zUai$yMKW>w&^mIynbxDT<%8NYkbn+=WbtTpd
zn2tNG@>jEk!6Zowu4Vg!X1r_sZ<Yl!HtxR7Fm$&9VIxmRXRPb32e@bfy#YelC1=Hh
zrOY`AXZ%m9Xzo|@Acq{kDiV-#o?E7k-3nvzPFZOTOMHW0K#kOy^TeEXJW4z7`^3L>
zUQ#zSJ@R;Ge#Rlz&GY`^0I4jplo^7CE8)(P<rR`_{_8^b5H$C}#;I4t#{3OWze@vd
z4OCpQv5H9O#F^i%@LJ9}UZc)a<!v7V_S-+2UM*(>W@TOg6wNG(*f4>^Gvd^%0_Rim
zMLdDO3Tew#k+6ThH~JGgrX6pTt6W|zoYb_86<%sioEw+Fs=vP}IYWWLdC?{P2`qn{
zoSbY<+TT*K2j2H^qoZv)CbiGCUKnVUiqcZgXh_0E4&`^;G}gB>6IFB&f*Udsqq0fw
zb8$G735vGk<M@+`ev>AJq|uph_sA&=N$UOk_eJban9N`Xqr~N1BI^cocI<S{qIZ~9
zdfoFnqnvClKKA`vkCjr^*yLUk%G%iU8HJf3OiTi<-{OvOsFvJF@9K8wmciK5Rm^3{
z!~{*Uq2Ks}wxHxPP!k-dgOV${7-6Sr0<~Q--s$fT&Izo)9~#!F(~{y8r{W|Rjbvc-
zB1n~<OqWhi3;gTSP=g|)oAs2`hK12WHgj>jeBNYP`d3s=?{&L2=TD_lJAZ8-i4_`L
zva3YeY<MLsh=z4u_b4lF9|A~9g&Ar2$_w&cLyEUj>y;57RKmh8k0tn+D9xYo=(2T<
z@BR?0uw!FNu5*Fat)!%+n3xz}obbbkhqZAXmB)Jb?D};KmlBj3nv?CG_MW+WZbQB}
z)mB5%<VU)Hm8FLBqx70Z-}yk@H}B@JY29#ENmRo`-eeAo#Xw*z{!?==<x(G1-KVj0
zpDHr@RM5=JgEfGE{~@2j9kam8Tk=Iia~Oj&Wc_PrZPQGZFck!jL^4{~7UfU`_P|^5
zm~8b%apJ8Si&_xAs=R~x$kMFSLvsf{sv)q-&a<m*9?Tud{uW%>vp20cO(*Wt&7s?M
zQ|}(ZrX4?KB<KcSZdXvfd-rZuiqebNGRWn={kuJEB1;aMh}%pe_@l>%W%GEX*OF1m
ze1YipPga!r`*~#c*j@&Q)4qfm(Bwm1z#8ZnL%z>`T;xGNHNSqagR(qZF*ogh;#j2m
z`-!h_=pIl(R{3J0lF^*4W(GS4V`=7juMl1wkl%OTKMhg!NM^$-kp4cu117F=C$8j;
z^laT$z{_ZBx|X$|6mmFrsDr?#nZ_V&N=vp5Qmjujx5~@QPjpw8Z<5U)@o*~1L_1%y
zTbX@SsoPD9O6|PikW)C2oTZqR?(u^jwQcIqsB4LofppJ>2yhm*7Ht02Ca~0^V%8&S
z_mu=}Gx#y^(vDgDR(zv9F)WUCJLhk_g(u-ENPZc4e+8EQ*h>)BagQsy&GrDwGQA#*
z{Bo-B^`f}Lw$Po8`RJXxS=^`|{Y-FIv`=7!KzO;}RY$iB%XS#j;rf~s+i>j4hx=E#
zOoP16k<&&_gqNJj!1;26r*-0fRRxobi9DA*gcrxL58YJ0yPqkQlp@S6a%EwdUz|Uf
zk00r-4w-72G9IoDE~j<pvSRG?wG`?T##$d2OjGzyG1HQwxPF%_rqjF7wA~N0-|Z;#
zOT%>c_7<kY-<=TB@_X2(t)3yf8%ce>moTboYTmtZ+T-Kn%hPRULd1SYks{w@?gH@w
zy(;(k<mt<NekwA`&C<`Wl#o_yQsGinQ?SJr`~$9SOhYR{%b}?vduL~`)(D^d5a&?%
zG2>k&T`(Cu-tdT3M~LQ!Cv^F@_BTBF9pOVdT4FewgfHzFwl~z4AFEP6%sSJKVk^oP
zx<@YuQ)6wF+-7AQqFOH`sF^YE#z&0Uij5U1dw3j0!PZZ1ypiA1fa7_{rHMR$#LCii
z+e#%oi0fK#EiuD3SmttII)~s{Lga77r)N8dQZEC+4R1YIB3||}jkW>t;xh1qUc6{H
zBI@4w`AecSX1-^$q+ukeXrvlj%yVmCg{~}xRnf@N(a}CWjb^C9Sgv%=*^+{bs~+T)
z4Ej=w`ZAZC%H5-8I=7iE*+rjjC36BYSSRcc*4}D8XOk4`P%7D4aB)-iol_OH@K}7W
zR!isGJ_%Dui*shqb%#hN5TQr;=vl+)b4LxI?XsvG$V8SN+ow}!wIfGe4$s&XKbzHY
z4z>|2e?4$9+i~ee?XQSCwJkkpqr<F=(O}L`Ax<;8LqiLA&Ec-j+?(D^x>7Q%;vvli
z<MbBnIJ=xQ`(QHMUf626a^PhZSZy%&Y~Cz999}gdPJ6HUlFP+SyAI^IGZ)^w<jAz`
zBU~6&?zqEgq72@?t@%o-rDJLVIFU@>H;aACi*vJeoaw%&F&Dzo$Hd6@jg5_g5blE_
zs5kuaB>&iZ&v^xlvAIRzUXr-h2^S`FF{ep^@vYv^&4)mPn?ZxIf#Ef2in1<yxvnyQ
z9{<-S=&wWaIurQ+W?ydtU*liT{{Lh`|F53Cq&@qx)@jZ<_J2CL>+`nfWI$O6Exb&m
SQ}*f$7;6h0mUYDA=f4A>?7Gtc

literal 0
HcmV?d00001