<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd"> 
<html xmlns="http://www.w3.org/1999/xhtml"> 
    <head> 
        <meta http-equiv="Content-Type" content="text/html; charset=utf-8" /> 
        <meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0" /> <base href="https://issues.jboss.org" /> 
        <title>Message Title</title> 
    </head> 
    <body class="jira" style="color: #333; font-family: Arial, sans-serif; font-size: 14px; line-height: 1.429"> 
        <table id="background-table" cellpadding="0" cellspacing="0" width="100%" style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt; background-color: #f5f5f5; border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt"> 
            <!-- header here --> 
            <tr> 
                <td id="header-pattern-container" style="padding: 0px; border-collapse: collapse; padding: 10px 20px"> 
                    <table id="header-pattern" cellspacing="0" cellpadding="0" border="0" style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt"> 
                        <tr> 
                            <td id="header-avatar-image-container" valign="top" style="padding: 0px; border-collapse: collapse; vertical-align: top; width: 32px; padding-right: 8px"> <img id="header-avatar-image" class="image_fix" src="https://community.jboss.org/gravatar/1ce36ce1c246b989f9cd79504c400b08?d=mm&amp;s=48" height="32" width="32" border="0" style="border-radius: 3px; vertical-align: top" /> 
                            </td> 
                            <td id="header-text-container" valign="middle" style="padding: 0px; border-collapse: collapse; vertical-align: middle; font-family: Arial, sans-serif; font-size: 14px; line-height: 20px; mso-line-height-rule: exactly; mso-text-raise: 1px"> <a class="user-hover" rel="lfryc" id="email_lfryc" href="https://issues.jboss.org/secure/ViewProfile.jspa?name=lfryc" style="color:#3b73af;; color: #3b73af; text-decoration: none">Luk&aacute;š Fryč</a> <strong>updated</strong> an issue 
                            </td> 
                        </tr> 
                    </table> 
                </td> 
            </tr> 
            <tr> 
                <td id="email-content-container" style="padding: 0px; border-collapse: collapse; padding: 0 20px"> 
                    <table id="email-content-table" cellspacing="0" cellpadding="0" border="0" width="100%" style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt; border-spacing: 0; border-collapse: separate"> 
                        <tr> 
                            <!-- there needs to be content in the cell for it to render in some clients --> 
                            <td class="email-content-rounded-top mobile-expand" style="padding: 0px; border-collapse: collapse; color: #fff; padding: 0 15px 0 16px; height: 15px; background-color: #fff; border-left: 1px solid #ccc; border-top: 1px solid #ccc; border-right: 1px solid #ccc; border-bottom: 0; border-top-right-radius: 5px; border-top-left-radius: 5px; height: 10px; line-height: 10px; padding: 0 15px 0 16px; mso-line-height-rule: exactly">
                                &nbsp;
                            </td> 
                        </tr> 
                        <tr> 
                            <td class="email-content-main mobile-expand " style="padding: 0px; border-collapse: collapse; border-left: 1px solid #ccc; border-right: 1px solid #ccc; border-top: 0; border-bottom: 0; padding: 0 15px 0 16px; background-color: #fff"> 
                                <table class="page-title-pattern" cellspacing="0" cellpadding="0" border="0" width="100%" style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt"> 
                                    <tr> 
                                        <td class="page-title-pattern-first-line " style="padding: 0px; border-collapse: collapse; font-family: Arial, sans-serif; font-size: 14px; padding-top: 10px"> <a href="https://issues.jboss.org/browse/AGPUSH" style="color: #3b73af; text-decoration: none">AeroGear Push</a> / <a href="https://issues.jboss.org/browse/AGPUSH-695" style="color: #3b73af; text-decoration: none"><img src="cid:jira-generated-image-static-health-87f3446c-e278-4038-8e78-2d9ac5b4e094" height="16" width="16" border="0" align="absmiddle" alt="Enhancement" style="vertical-align: text-bottom" /></a> <a href="https://issues.jboss.org/browse/AGPUSH-695" style="color: #3b73af; text-decoration: none">AGPUSH-695</a> 
                                        </td> 
                                    </tr> 
                                    <tr> 
                                        <td style="vertical-align: top;; padding: 0px; border-collapse: collapse; padding-right: 5px; font-size: 20px; line-height: 30px; mso-line-height-rule: exactly" class="page-title-pattern-header-container"> <span class="page-title-pattern-header" style="font-family: Arial, sans-serif; padding: 0; font-size: 20px; line-height: 30px; mso-text-raise: 2px; mso-line-height-rule: exactly; vertical-align: middle"> <a href="https://issues.jboss.org/browse/AGPUSH-695" style="color: #3b73af; text-decoration: none">Admin UI: apply findings of Code Style review</a> </span> 
                                        </td> 
                                    </tr> 
                                </table> 
                            </td> 
                        </tr> 
                        <tr> 
                            <td class="email-content-main mobile-expand  wrapper-special-margin" style="padding: 0px; border-collapse: collapse; border-left: 1px solid #ccc; border-right: 1px solid #ccc; border-top: 0; border-bottom: 0; padding: 0 15px 0 16px; background-color: #fff; padding-top: 10px; padding-bottom: 5px"> 
                                <table class="keyvalue-table" style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt"> 
                                    <tr> 
                                        <th style="color: #707070; font: normal 14px/20px Arial, sans-serif; text-align: left; vertical-align: top; padding: 2px 0">
                                            Change By:
                                        </th> 
                                        <td style="padding: 0px; border-collapse: collapse; font: normal 14px/20px Arial, sans-serif; padding: 2px 0 2px 5px; vertical-align: top"> <a class="user-hover" rel="lfryc" id="email_lfryc" href="https://issues.jboss.org/secure/ViewProfile.jspa?name=lfryc" style="color:#3b73af;; color: #3b73af; text-decoration: none">Luk&aacute;š Fryč</a> 
                                        </td> 
                                    </tr> 
                                </table> 
                            </td> 
                        </tr> 
                        <tr> 
                            <td class="email-content-main mobile-expand  issue-description-container" style="padding: 0px; border-collapse: collapse; border-left: 1px solid #ccc; border-right: 1px solid #ccc; border-top: 0; border-bottom: 0; padding: 0 15px 0 16px; background-color: #fff; padding-top: 5px; padding-bottom: 10px"> 
                                <table class="text-paragraph-pattern" cellspacing="0" cellpadding="0" border="0" width="100%" style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt; font-family: Arial, sans-serif; font-size: 14px; line-height: 20px; mso-line-height-rule: exactly; mso-text-raise: 2px"> 
                                    <tr> 
                                        <td class="text-paragraph-pattern-container mobile-resize-text " style="padding: 0px; border-collapse: collapse; padding: 0 0 10px 0"> <span class="diffcontext">Forum&nbsp;discussion:<br />http://aerogear-dev.1069024.n5.nabble.com/aerogear-dev-AG-Push-Console-review-td8004.html<br /><br />----<br /><br /></span> <span class="diffremovedchars" style="background-color: #ffe7e7; text-decoration:line-through;">*</span> <span class="diffaddedchars" style="background-color:#ddfade;">{quote}<br /></span> <span class="diffcontext">&nbsp;The&nbsp;name&nbsp;of&nbsp;your&nbsp;angularjs&nbsp;module&nbsp;is&nbsp;&quot;newadminApp&quot;.&nbsp;I&nbsp;think&nbsp;it&nbsp;should<br />be&nbsp;something&nbsp;like&nbsp;&quot;agconsole&quot;.<br /></span> <span class="diffaddedchars" style="background-color:#ddfade;">{quote}</span> <span class="diffcontext"><br /></span> <span class="diffremovedchars" style="background-color: #ffe7e7; text-decoration:line-through;">*</span> <span class="diffaddedchars" style="background-color:#ddfade;"><br />created&nbsp;AGPUSH-719<br /><br />{quote}<br /></span> <span class="diffcontext">&nbsp;You're&nbsp;missing&nbsp;feedback&nbsp;for&nbsp;user&nbsp;on&nbsp;many&nbsp;places.&nbsp;If&nbsp;the&nbsp;error&nbsp;says<br />'Something&nbsp;went&nbsp;wrong...',&nbsp;it&nbsp;doesn't&nbsp;say&nbsp;much&nbsp;to&nbsp;a&nbsp;user.&nbsp;Even&nbsp;changing<br />it&nbsp;to&nbsp;denote&nbsp;the&nbsp;current&nbsp;operation&nbsp;would&nbsp;be&nbsp;better,&nbsp;something&nbsp;like<br />'Unable&nbsp;to&nbsp;create&nbsp;new&nbsp;application'.&nbsp;On&nbsp;most&nbsp;of&nbsp;the&nbsp;$resource.*&nbsp;methods<br />you're&nbsp;missing&nbsp;the&nbsp;feedback,&nbsp;too.&nbsp;For&nbsp;most&nbsp;of&nbsp;them&nbsp;the&nbsp;information&nbsp;about<br />success&nbsp;would&nbsp;be&nbsp;annoying,&nbsp;but&nbsp;maybe&nbsp;it&nbsp;could&nbsp;be&nbsp;interesting&nbsp;for&nbsp;user&nbsp;to<br />see&nbsp;errors&nbsp;like&nbsp;'Unable&nbsp;to&nbsp;get&nbsp;the&nbsp;application&nbsp;list,&nbsp;check&nbsp;your<br />connection&nbsp;to&nbsp;the&nbsp;server...'.&nbsp;When&nbsp;my&nbsp;session&nbsp;to&nbsp;the&nbsp;console&nbsp;timed-out,<br />the&nbsp;operation&nbsp;I&nbsp;was&nbsp;trying&nbsp;to&nbsp;do&nbsp;(renaming&nbsp;of&nbsp;app)&nbsp;didn't&nbsp;do&nbsp;anything,<br />but&nbsp;I&nbsp;had&nbsp;no&nbsp;clue&nbsp;why.<br /></span> <span class="diffaddedchars" style="background-color:#ddfade;">{quote}</span> <span class="diffcontext"><br /></span> <span class="diffremovedchars" style="background-color: #ffe7e7; text-decoration:line-through;">*</span> <span class="diffaddedchars" style="background-color:#ddfade;"><br />created&nbsp;AGPUSH-720<br /><br />{quote}<br /></span> <span class="diffcontext">&nbsp;AFAIK&nbsp;it's&nbsp;considered&nbsp;a&nbsp;bad&nbsp;practice&nbsp;to&nbsp;use&nbsp;jQuery&nbsp;inside&nbsp;controllers.<br />You&nbsp;use&nbsp;only&nbsp;two&nbsp;$&nbsp;functions:&nbsp;$.extend&nbsp;and&nbsp;$.ajax,&nbsp;which&nbsp;can&nbsp;be&nbsp;replaced<br />with&nbsp;angular.extend/angular.copy&nbsp;and&nbsp;the&nbsp;$resource&nbsp;service.&nbsp;If&nbsp;you&nbsp;can't<br />find&nbsp;a&nbsp;way&nbsp;how&nbsp;to&nbsp;do&nbsp;something&nbsp;without&nbsp;jQuery&nbsp;inside&nbsp;controller,&nbsp;maybe<br />creating&nbsp;of&nbsp;custom&nbsp;directive&nbsp;is&nbsp;the&nbsp;right&nbsp;solution,&nbsp;but&nbsp;this&nbsp;doesn't<br />seem&nbsp;to&nbsp;be&nbsp;the&nbsp;case.<br /></span> <span class="diffaddedchars" style="background-color:#ddfade;">{quote}</span> <span class="diffcontext"><br /></span> <span class="diffremovedchars" style="background-color: #ffe7e7; text-decoration:line-through;">*</span> <span class="diffaddedchars" style="background-color:#ddfade;"><br />created&nbsp;AGPUSH-721<br /><br />{quote}<br /></span> <span class="diffcontext">&nbsp;It&nbsp;seems&nbsp;that&nbsp;you're&nbsp;using&nbsp;several&nbsp;switch&nbsp;statements&nbsp;just&nbsp;to&nbsp;assign&nbsp;a<br />value&nbsp;based&nbsp;on&nbsp;input.&nbsp;This&nbsp;seems&nbsp;to&nbsp;be&nbsp;a&nbsp;good&nbsp;candidate&nbsp;for&nbsp;using&nbsp;a&nbsp;map.<br /></span> <span class="diffaddedchars" style="background-color:#ddfade;">{quote}</span> <span class="diffcontext"><br /></span> <span class="diffaddedchars" style="background-color:#ddfade;"><br />I&nbsp;don't&nbsp;consider&nbsp;this&nbsp;bad&nbsp;practice&nbsp;since&nbsp;it&nbsp;explains&nbsp;well&nbsp;that&nbsp;some&nbsp;options&nbsp;has&nbsp;same&nbsp;value&nbsp;as&nbsp;given&nbsp;key:<br />https://github.com/aerogear/aerogear-unifiedpush-server/blob/f2c5a5d7d834cc74e4a66bcb966437d2de68981d/admin-ui/app/scripts/controllers/DetailController.js#L186<br /><br /><br /></span> <span class="diffcontext">*&nbsp;In&nbsp;installation.html&nbsp;I&nbsp;see&nbsp;stuff&nbsp;like&nbsp;&lt;col&nbsp;width=&quot;45%&quot;&gt;.&nbsp;I&nbsp;think&nbsp;this<br />should&nbsp;be&nbsp;rewritten&nbsp;to&nbsp;use&nbsp;the&nbsp;bootstrap&nbsp;grid&nbsp;layout&nbsp;(classes&nbsp;like<br />col-md-12...).<br /><br />*&nbsp;We&nbsp;use&nbsp;our&nbsp;custom&nbsp;directive&nbsp;lo-autofocus&nbsp;for&nbsp;the&nbsp;1st&nbsp;input&nbsp;in&nbsp;modals.<br />Right&nbsp;after&nbsp;the&nbsp;modal&nbsp;pops-up,&nbsp;user&nbsp;has&nbsp;the&nbsp;focus&nbsp;on&nbsp;the&nbsp;first&nbsp;input&nbsp;and<br />pressing&nbsp;enter&nbsp;submits&nbsp;the&nbsp;modal.<br /><br />*&nbsp;The&nbsp;last&nbsp;one&nbsp;is&nbsp;not&nbsp;a&nbsp;problem&nbsp;at&nbsp;all,&nbsp;I&nbsp;just&nbsp;don't&nbsp;feel&nbsp;right&nbsp;about<br />it&nbsp;:)&nbsp;You&nbsp;use&nbsp;$window.location.href&nbsp;instead&nbsp;$location.absUrl().&nbsp;They<br />seem&nbsp;to&nbsp;do&nbsp;the&nbsp;same,&nbsp;but&nbsp;since&nbsp;you&nbsp;don't&nbsp;use&nbsp;any&nbsp;other&nbsp;$window&nbsp;stuff,<br />using&nbsp;$location&nbsp;makes&nbsp;more&nbsp;sense&nbsp;to&nbsp;me.<br /><br />----<br /><br />Addressed&nbsp;by&nbsp;other&nbsp;issues:<br /><br />*&nbsp;AGPUSH-659:&nbsp;Notifications&nbsp;-&nbsp;they&nbsp;are&nbsp;inside&nbsp;ng-view&nbsp;pages.&nbsp;It&nbsp;looks&nbsp;fine&nbsp;and&nbsp;works<br />for&nbsp;your&nbsp;current&nbsp;purposes.&nbsp;But&nbsp;if&nbsp;you&nbsp;make&nbsp;some&nbsp;new&nbsp;page&nbsp;in&nbsp;the&nbsp;future,<br />which&nbsp;redirects&nbsp;you&nbsp;(i.e.&nbsp;creating&nbsp;a&nbsp;new&nbsp;entity,&nbsp;after&nbsp;submit,&nbsp;user&nbsp;is<br />redirected&nbsp;to&nbsp;entity&nbsp;list&nbsp;with&nbsp;newly&nbsp;created&nbsp;stuff),&nbsp;you&nbsp;may&nbsp;loose&nbsp;the<br />message.&nbsp;Another&nbsp;disadvantage&nbsp;of&nbsp;this&nbsp;is&nbsp;non-consistent&nbsp;L&amp;F&nbsp;for&nbsp;feedback<br />messages.&nbsp;The&nbsp;advantage&nbsp;is,&nbsp;that&nbsp;you're&nbsp;not&nbsp;covering&nbsp;any&nbsp;content&nbsp;with<br />the&nbsp;message&nbsp;as&nbsp;LO&nbsp;does.</span> 
                                        </td> 
                                    </tr> 
                                </table> 
                            </td> 
                        </tr> 
                        <tr> 
                            <td class="email-content-main mobile-expand " style="padding: 0px; border-collapse: collapse; border-left: 1px solid #ccc; border-right: 1px solid #ccc; border-top: 0; border-bottom: 0; padding: 0 15px 0 16px; background-color: #fff"> 
                                <table id="actions-pattern" cellspacing="0" cellpadding="0" border="0" width="100%" style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt; font-family: Arial, sans-serif; font-size: 14px; line-height: 20px; mso-line-height-rule: exactly; mso-text-raise: 1px"> 
                                    <tr> 
                                        <td id="actions-pattern-container" valign="middle" style="padding: 0px; border-collapse: collapse; padding: 10px 0 10px 24px; vertical-align: middle; padding-left: 0"> 
                                            <table align="left" style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt"> 
                                                <tr> 
                                                    <td class="actions-pattern-action-icon-container" style="padding: 0px; border-collapse: collapse; font-family: Arial, sans-serif; font-size: 14px; line-height: 20px; mso-line-height-rule: exactly; mso-text-raise: 0px; vertical-align: middle"> <a href="https://issues.jboss.org/browse/AGPUSH-695#add-comment" target="_blank" title="Add Comment" style="color: #3b73af; text-decoration: none"> <img class="actions-pattern-action-icon-image" src="cid:jira-generated-image-static-comment-icon-cd8edbbe-dfd4-4fef-b866-84ee923ba2cd" alt="Add Comment" title="Add Comment" height="16" width="16" border="0" style="vertical-align: middle" /> </a> 
                                                    </td> 
                                                    <td class="actions-pattern-action-text-container" style="padding: 0px; border-collapse: collapse; font-family: Arial, sans-serif; font-size: 14px; line-height: 20px; mso-line-height-rule: exactly; mso-text-raise: 4px; padding-left: 5px"> <a href="https://issues.jboss.org/browse/AGPUSH-695#add-comment" target="_blank" title="Add Comment" style="color: #3b73af; text-decoration: none">Add Comment</a> 
                                                    </td> 
                                                </tr> 
                                            </table> 
                                        </td> 
                                    </tr> 
                                </table> 
                            </td> 
                        </tr> 
                        <!-- there needs to be content in the cell for it to render in some clients --> 
                        <tr> 
                            <td class="email-content-rounded-bottom mobile-expand" style="padding: 0px; border-collapse: collapse; color: #fff; padding: 0 15px 0 16px; height: 5px; line-height: 5px; background-color: #fff; border-top: 0; border-left: 1px solid #ccc; border-bottom: 1px solid #ccc; border-right: 1px solid #ccc; border-bottom-right-radius: 5px; border-bottom-left-radius: 5px; mso-line-height-rule: exactly">
                                &nbsp;
                            </td> 
                        </tr> 
                    </table> 
                </td> 
            </tr> 
            <tr> 
                <td id="footer-pattern" style="padding: 0px; border-collapse: collapse; padding: 12px 20px"> 
                    <table id="footer-pattern-container" cellspacing="0" cellpadding="0" border="0" style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt"> 
                        <tr> 
                            <td id="footer-pattern-text" class="mobile-resize-text" width="100%" style="padding: 0px; border-collapse: collapse; color: #999; font-size: 12px; line-height: 18px; font-family: Arial, sans-serif; mso-line-height-rule: exactly; mso-text-raise: 2px">
                                 This message was sent by Atlassian JIRA <span id="footer-build-information">(v6.2.6#6264-<span title="ee7642271310c09537d01e5848a003c4498a0eed" data-commit-id="ee7642271310c09537d01e5848a003c4498a0eed}">sha1:ee76422</span>)</span> 
                            </td> 
                            <td id="footer-pattern-logo-desktop-container" valign="top" style="padding: 0px; border-collapse: collapse; padding-left: 20px; vertical-align: top"> 
                                <table style="border-collapse: collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt"> 
                                    <tr> 
                                        <td id="footer-pattern-logo-desktop-padding" style="padding: 0px; border-collapse: collapse; padding-top: 3px"> <img id="footer-pattern-logo-desktop" src="cid:jira-generated-image-static-footer-desktop-logo-2cad37f7-ac90-4970-acaf-a25b0500644f" alt="Atlassian logo" title="Atlassian logo" width="169" height="36" class="image_fix" /> 
                                        </td> 
                                    </tr> 
                                </table> 
                            </td> 
                        </tr> 
                    </table> 
                </td> 
            </tr> 
        </table>   
    </body>
</html>