PR

【今日中】アバター画像のhttps置換エラー対応 #730

wada opened this at 2021-04-12T11:49:09; closed

Description

opened by wada at 2021-04-12T11:49:09; updated 2024-05-09T11:43:49

引数がnullの時にエラーが出ていた

文字列の場合だけ置換するように修正
また開発環境だとhttpsで画像を取得できないので、置換しないように修正

Comment

#10254 by wada at 2021-04-12T11:50:35; updated 2021-04-12T11:50:35

@kouho @iwamoto
レビューお願いします。
nullや数値、変な文字列で置換されないことを確認できればOKです。
ローカル環境だと、TEST判定されて置換されないと思うので、そこの処理だけコメントアウトしておいてください。

Comment

#10259 by iwamoto at 2021-04-12T12:22:05; updated 2021-04-12T12:22:05

@wada
こちらレビューしました。
「nullや数値、変な文字列で置換されないこと」を確認しました。
問題ないと思います。

Comment

#10264 by kouho at 2021-04-12T16:14:02; updated 2021-04-12T16:14:02

@wada
レビューしました。
「nullや数値、変な文字列で置換されないこと」を確認いたしました。
問題ないかと思います。

kouho add_label at 2021-04-12T16:14:06: 皇甫 LGTM
wada add_label at 2021-04-12T16:45:16: 岩本 LGTM
wada merge at 2021-04-12T16:45:27: 【今日中】アバター画像のhttps置換エラー対応
wada close at 2021-04-12T16:45:27: Close

Commit range

base82281664d3a34f0b59a4c47d1014dcc74e9fe888
head9fe9e691a14d2a4a48a0bf800b201e604c26d6bc
base branchjolly/jr-web:develop
head branchjolly/jr-web:fix_replace_avatar

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed1 filesView patch
statusfile
modifiedlib/SubFunc.inc
Patch body is stored as a separate file to keep this page lightweight.