{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
-
Toggle commit list
-
-
151 110 } 152 111 } 153 112 113 /* -
Master
[nits] こちらからコメントアウトされています。確認お願いします。
-
Master
削除します。
-
-
-
172 127 pushMsgDialog.putExtra(AppDefType.ChatPushMessageKey.roomId, roomId); 173 128 pushMsgDialog.putExtra(AppDefType.ChatPushMessageKey.roomName, roomName); 174 pushMsgDialog.putExtra(AppDefType.ChatPushMessageKey.roomType, roomType); 175 129 pushMsgDialog.putExtra(AppDefType.ChatPushMessageKey.pushSendLoginId, pushSendLoginId); 176 130 pushMsgDialog.putExtra(AppDefType.ChatPushMessageKey.pushSendDate, pushSendDate); 177 131 pushMsgDialog.putExtra(AppDefType.ChatPushMessageKey.collaborationType, collaborationType); 178 pushMsgDialog.putExtra(AppDefType.ChatPushMessageKey.fromClassName, fromClassName); 179 132 pushMsgDialog.putExtra(AppDefType.ChatPushMessageKey.roomType, roomType); 180 133 startActivity(pushMsgDialog); 181 134 } 182 135 */ 183 136 /** 184 137 * プッシュメッセージをダイアログ表示する 185 * @param textMessage プッシュメッセージ -
211 152 return; 212 153 } 213 154 } 155 214 156 String fromClassName = currentActivity.getClass().getName(); 215 157 if (currentActivity.getClass().equals(HTMLWebViewActivity.class) || 216 158 currentActivity.getClass().equals(ContentViewActivity.class) || 217 159 currentActivity.getClass().equals(OperationRelatedContentActivity.class) || 218 currentActivity.getClass().equals(OperationListActivity.class) 160 currentActivity.getClass().equals(OperationListActivity.class) || 161 currentActivity.getClass().equals(ImageViewActivity.class) || 162 currentActivity.getClass().equals(VideoViewActivity.class) 219 163 ){ 220 ((ABVAuthenticatedActivity)currentActivity).showChatRoomPopupMessage((Context)currentActivity, textMessage, 221 data, operationID, roomID, roomName, roomType, pushSendLoginId, pushSendDate, collaborationType, fromClassName); 164 // 呼び出し元のActivity名を保存しておく -
Master
[nits]
Activityをどこに保存するのかコードでは見当たらないです。コードを読んだ限りだと、messagemapに格納し他の関数に渡しているように見えます。
-
-
-
215 157 if (currentActivity.getClass().equals(HTMLWebViewActivity.class) || 216 158 currentActivity.getClass().equals(ContentViewActivity.class) || 217 159 currentActivity.getClass().equals(OperationRelatedContentActivity.class) || 218 currentActivity.getClass().equals(OperationListActivity.class) 160 currentActivity.getClass().equals(OperationListActivity.class) || 161 currentActivity.getClass().equals(ImageViewActivity.class) || 162 currentActivity.getClass().equals(VideoViewActivity.class) 219 163 ){ 220 ((ABVAuthenticatedActivity)currentActivity).showChatRoomPopupMessage((Context)currentActivity, textMessage, 221 data, operationID, roomID, roomName, roomType, pushSendLoginId, pushSendDate, collaborationType, fromClassName); 164 // 呼び出し元のActivity名を保存しておく 165 messageMap.put(AppDefType.ChatPushMessageKey.fromActivityName, fromClassName); 166 messageMap.put(AppDefType.ChatPushMessageKey.isNewVersion, "true"); 167 ((ABVAuthenticatedActivity)currentActivity).showChatRoomPopupMessage((Context)currentActivity, messageMap); 222 168 } else { 169 // 既存の処理 -
Master
[nits] 既読の処理が見当たらないです。pushmsgdialog intent(?)を作成し、そこにデータを格納し、新しいactivityを開く処理に見えます。
-
Master
コメントの間違いでした。 「それ以外のActivityの場合の処理」に修正します。
-
-
-
318 /** 319 * 通知エリアに表示するテキストを決定する 320 * @param textMessage プッシュメッセージをJSONOjbcetにした時の、AppDefType.PushMessageKey.message の値 321 * @return 表示するテキスト 322 */ 323 private String getContentText(String textMessage) { 324 if (!StringUtil.isNullOrEmpty(textMessage) && textMessage.contains(ABookCommConstants.INVITE_COLLABORATION)) { 325 return getString(R.string.msg_invite_collaboration); 326 } 327 return textMessage; 328 } 329 330 /** 331 * プッシュメッセージからJSONObjectを作成する。 332 * @param jsonText 受信したプッシュメッセージをJSONObjectにする。 333 * @return -
Master
returnの値があるのにコメントが書いてないです。
-
Master
コメントを追加します。
-
-
-
320 * @param textMessage プッシュメッセージをJSONOjbcetにした時の、AppDefType.PushMessageKey.message の値 321 * @return 表示するテキスト 322 */ 323 private String getContentText(String textMessage) { 324 if (!StringUtil.isNullOrEmpty(textMessage) && textMessage.contains(ABookCommConstants.INVITE_COLLABORATION)) { 325 return getString(R.string.msg_invite_collaboration); 326 } 327 return textMessage; 328 } 329 330 /** 331 * プッシュメッセージからJSONObjectを作成する。 332 * @param jsonText 受信したプッシュメッセージをJSONObjectにする。 333 * @return 334 */ 335 private JSONObject createMessageJsonObject(String jsonText) { -
Master
[ask] MRの中では、こちらの関数が使用されているところがなさそうに見えます。。
-
Master
削除します。
-
-
-
1244 1243 } 1245 1244 1246 1245 /** 1247 * プッシュメッセージ受信後のダイアログを表示する。 1248 * @param context コンテキスト 1249 * @param textMessage プッシュメッセージ 1250 * @param data Intentに付属するデータ 1251 * @param operationID operationID 1252 * @param roomID roomID 1253 * @param roomName roomName 1254 * @param roomType roomType 1255 * @param pushSendLoginId pushSendLoginId 1256 * @param pushSendDate pushSendDate 1257 * @param collaborationType ABookCommConstants.COLLABORATION_TYPE のいずれか 1246 * ッシュメッセージがあるので、チャットルームに遷移する。 -
Master
ッシュ → プッシュ
-
Master
修正します。
-
-
-
1292 * チャットルームへ遷移する。 1281 * プッシュメッセージ受信後のダイアログを表示する。 1282 * @param context コンテキスト 1283 * @param messageMap プッシュメッセージ 1293 1284 */ 1294 public void goChatRoom() { 1295 if (roomId <= 0 && StringUtil.isNullOrEmpty(roomName)) { 1285 public void showChatRoomPopupMessage(Context context, Map<String, String> messageMap) { 1286 String messageBody = messageMap.get(AppDefType.PushMessageKey.message); 1287 PushMessageJSON json; 1288 if (StringUtil.isNullOrEmpty(messageBody)) { 1296 1289 return; 1297 1290 } 1298 if (StringUtil.isNullOrEmpty(collaborationType)) { 1299 ActivityHandlingHelper.getInstance().startChatWebViewActivity(fromClassName, roomId, roomName, roomType); 1291 json = new PushMessageJSON(messageBody); -
125 125 isCollabration = MeetingManager.getInstance().isCollaboration(); 126 126 Bundle extras = getIntent().getExtras(); 127 127 if (extras != null && !isMeetingRoomConnected) { 128 setResult(ABookCommConstants.PUSH_MESSAGE_DLG_RESULT.OK, getIntent()); 129 finish(); 128 boolean isNewVersion = extras.getBoolean(AppDefType.ChatPushMessageKey.isNewVersion); -
Master
[ask] chatにnew messageバージョンが存在していたのかこんがらがるのですが、こちらはどういう新しいバージョンをチェックしているのでしょうか。
-
Master
isNewVersion がtrueの場合は、iOSと同じ画面遷移をします。falseの場合は、いままでと同じ画面遷移をします。
-
-
-
159 161 int PAYMENT = 3; 160 162 } 161 163 162 // ABookCommunicationで使用する。 163 private Long roomId = new Long(0); 164 private String roomName = ""; 165 private String collaborationType = ""; 166 private String roomType = ""; 167 private String fromClassName = ""; 164 // 遷移元のアクティビティ名 165 // PushMessage受信時に直接ChatRoomへ行かずに、ひとつまえのActivityに戻る為に使用する。 166 public String fromActivityName; -
Master
fromActivityName->sourceActivityNameorbaseActivityName -
Master
baseActivityNameにします。
-
-
-
1697 1698 intent.setClassName(mContext.getPackageName(), nextActivity); 1699 intent.setFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP); 1700 getCurrentActivity().startActivity(intent); 1701 } 1702 1703 /** 1704 1672 * プシュメッセージからチャットに遷移する 1705 * fromClassName の値によっては、作業一覧、コンテンツ一覧のどちらかに戻ってから、チャットに遷移する） 1706 * @param fromClassName 呼び出し元のクラス名（ 1707 * @param roomId roomId 1708 * @param roomName room 1709 * @param roomType roomaType 1710 * @param collaborationType ABookCommConstants.COLLABORATION_TYPE のいずれか 1673 * @param roomId 1674 * @param roomName -
Master
[nits] コメントに
String nextActivityName, String fromActivityNameの説明がないです。 -
Master
説明を追加します。
-
-
-
1701 } 1702 1703 /** 1704 1672 * プシュメッセージからチャットに遷移する 1705 * fromClassName の値によっては、作業一覧、コンテンツ一覧のどちらかに戻ってから、チャットに遷移する） 1706 * @param fromClassName 呼び出し元のクラス名（ 1707 * @param roomId roomId 1708 * @param roomName room 1709 * @param roomType roomaType 1710 * @param collaborationType ABookCommConstants.COLLABORATION_TYPE のいずれか 1673 * @param roomId 1674 * @param roomName 1711 1675 */ 1712 public void startChatWebViewActivityWithCollaboration(String fromClassName, Long roomId, String roomName, String roomType, String collaborationType) { 1713 String className = ChatWebViewActivity.class.getName(); 1676 public void startChatWebViewActivity(Long roomId, String roomName, String nextActivityName, String fromActivityName) { -
Master
修正しました。
-
-
1965 1917 } 1966 1918 1967 1919 /** 1920 * プッシュメッセージから協業に遷移する 1921 * @param roomId 1922 * @param roomName 1923 * @param collaborationType 1924 * @param roomType 1925 */ 1926 public void startChatWebViewActivityWithCollaboration(Long roomId, String roomName, String collaborationType, String roomType, String nextActivityName, String fromActivityName) { 1927 boolean isNormalSize = (mContext.getResources().getConfiguration().screenLayout & Configuration.SCREENLAYOUT_SIZE_MASK) == Configuration.SCREENLAYOUT_SIZE_NORMAL; 1928 Intent intent = new Intent(); 1929 intent.putExtra("chatWebviewUrl",ABVEnvironment.getInstance().acmsAddress + ABVDataCache.getInstance().getUrlPath() + "/chatapi/chat/"); -
Master
[nits]
,の後はスペースを開けてください。 -
Master
開けました
-
-
-
1965 1917 } 1966 1918 1967 1919 /** 1920 * プッシュメッセージから協業に遷移する 1921 * @param roomId 1922 * @param roomName 1923 * @param collaborationType 1924 * @param roomType 1925 */ 1926 public void startChatWebViewActivityWithCollaboration(Long roomId, String roomName, String collaborationType, String roomType, String nextActivityName, String fromActivityName) { 1927 boolean isNormalSize = (mContext.getResources().getConfiguration().screenLayout & Configuration.SCREENLAYOUT_SIZE_MASK) == Configuration.SCREENLAYOUT_SIZE_NORMAL; -
Master
[must] 宣言されているけど、使われているところがございません。
-
Master
削除しました。他にも同じ処理があったので削除しました。
-
-
-
3829 3828 3830 3829 if (requestCode == ABookCommConstants.PUSH_MESSAGE_DLG_REQUEST_CODE) { 3831 if (resultCode == ABookCommConstants.PUSH_MESSAGE_DLG_RESULT.OK) { 3832 // Activityを閉じて遷移する 3833 exitActivity(); 3834 goChatRoom(); 3830 // プッシュメッセージダイアログのリザルトだった場合 3831 if (resultCode == ABookCommConstants.PUSH_MESSAGE_DLG_RESULT.OK && !StringUtil.isNullOrEmpty(fromActivityName)) { 3832 finishActivity(); 3833 goChatRoom(intent, fromActivityName, ContentViewActivity.class.getName()); 3835 3834 } 3836 3835 return; 3837 3836 } 3838 3837 3839 if (intent != null && resultCode == RESULT_OK) { 3838 if (intent != null && resultCode == RESULT_OK) { -
Master
[nits] indentの確認をお願いします。他の行ではスペースを使っています。
-
Master
スペースに変更しました。ここだけではなく既存のソースコードはタブになっています。
-
-
-
3851 3850 isAnotherViewOpenFlg = false; 3852 3851 // playPageBGMSound(mCurrentPageNumber); //프리뷰 모드 일때 음악재생 Bug 3853 3852 break; 3854 case ABookCommConstants.ABOOK_CHECK_TASK_IMAGE: 3853 case ABookCommConstants.ABOOK_CHECK_TASK_IMAGE: -
Master
[nits] indentを確認してください。
-
Master
スペースにしました。
-
-
-
98 98 msg.put(AppDefType.PushMessageKey.message, textMessage); 99 99 } 100 100 101 String roomType = null; 102 String[] inviteMessage; 101 103 if (textMessage.contains(INVITE_COLLABORATION)) { 102 String roomType = Integer.toString(json.getInt(AppDefType.ChatPushMessageKey.roomType)); 103 String[] inviteMessage = textMessage.split(CHAT_MESSAGE_SEPERATOR); 104 showCollaborationPopUpMessage(roomId, roomName, roomType, pushSendLoginId, pushSendDate, inviteMessage[1]); 105 return; 104 roomType = Integer.toString(json.getInt(AppDefType.ChatPushMessageKey.roomType)); 105 inviteMessage = textMessage.split(CHAT_MESSAGE_SEPERATOR); 106 //showCollaborationPopUpMessage(roomId, roomName, roomType, pushSendLoginId, pushSendDate, inviteMessage[1]); 107 //return; 108 Logger.i(TAG,"" + roomType); 109 Logger.i(TAG,"" + inviteMessage[1]); -
-
-
1295 messageText = getString(R.string.msg_invite_collaboration); 1296 } 1297 1298 Intent pushMsgDialog = new Intent(context, ShowPushMessageDailogActivity.class); 1299 pushMsgDialog.putExtra(AppDefType.PushMessageKey.message, messageText); 1300 pushMsgDialog.putExtra(AppDefType.PushMessageKey.data, json.getData()); 1301 pushMsgDialog.putExtra(AppDefType.PushMessageKey.operationId, json.getOperationID()); 1302 pushMsgDialog.putExtra(AppDefType.ChatPushMessageKey.roomId, json.getRoomId()); 1303 pushMsgDialog.putExtra(AppDefType.ChatPushMessageKey.roomName, json.getRoomName()); 1304 pushMsgDialog.putExtra(AppDefType.ChatPushMessageKey.pushSendLoginId, json.getPushSendLoginId()); 1305 pushMsgDialog.putExtra(AppDefType.ChatPushMessageKey.pushSendDate, json.getPushSendDate()); 1306 pushMsgDialog.putExtra(AppDefType.ChatPushMessageKey.roomType, json.getRoomType()); 1307 pushMsgDialog.putExtra(AppDefType.ChatPushMessageKey.collaborationType, json.getCollaborationType()); 1308 1309 pushMsgDialog.putExtra(AppDefType.ChatPushMessageKey.fromActivityName, messageMap.get(AppDefType.ChatPushMessageKey.fromActivityName)); 1310 String newVersion = messageMap.get(AppDefType.ChatPushMessageKey.needsDisplayOperationOrOperationRelatedContentScreen); -
merged
Toggle commit list
