Qiitaにずいぶん前に投稿したこの記事、結局どんなバグでいつ修正されたのか確認してないやつなんだよなぁって見返してた。httpsのphp特有なのかよくわからなかったWebKitのロケーションヘッダーのバグ
https://qiita.com/kPherox/items/37b755d17eb0e22d70fe
Qiitaにずいぶん前に投稿したこの記事、結局どんなバグでいつ修正されたのか確認してないやつなんだよなぁって見返してた。httpsのphp特有なのかよくわからなかったWebKitのロケーションヘッダーのバグ
https://qiita.com/kPherox/items/37b755d17eb0e22d70fe
@kPherox テンプレートの方、もしかしてWindows版だと取得できないとかあるかしら？もしそうだったらundefinedkHzになるから書き直しいるだろうか
@toneji それぞれ内容は別物ですがアートワーク添付の機能に関係があるのは34の方ですね。私が開発中に同じ画像を複数添付するバグが発生したので修正したものです
35は{hz}をサンプルレートの値に変換していなかったので修正した感じですね
@kPherox npmパッケージの方が公開されたら別ブランチ切って修正することにしよう
@kPherox アートワーク関係のないものとか書いてる時に見つけた不具合は以下で修正
Fix duplicate listener for iTunes NowPlaying
https://github.com/cutls/TheDesk/pull/34
Fix NowPlaying template for macOS
https://github.com/cutls/TheDesk/pull/35
@kPherox （Windows版を全く考慮に入れてなかった）
TheDeskのWIPでNowPlayingにアートワークつける機能をWindowsと共存させる形にしてくれてるので私のforkの方は更新しなくて良さそうかな？
TheDeskで開いたPR、WIPブランチじゃなくてmasterブランチがベースになってるから直しとこう
NowPlayingの文字列置き換えの処理、なんかムズムズする
@kPherox あー、あと{hz}が変換されない問題も別ブランチにしておくか。関係ないし
iTunesのNowPlayingでイベントリスナが残ったままになって複数回NowPlayingの取得が発生していたのを修正するPR
https://github.com/cutls/TheDesk/pull/34
アートワーク添付する機能を実装してる時にこれのせいで同じアートワークを何度もアップロードするので同じ画像が4枚も5枚も追加されてしまってたので
私の変更関係なくitunes-npのリスナ残ってる。取得を何度もする必要はないしonceに置き換えで良さそう。itunes以外は大丈夫なのかな？
@kPherox 今回の変更が関係なさそうなら別のブランチ切ろうね（全部同じブランチで作業してしまう悪癖を意識的に直していく
itunesのリスナ呼んだ時にitunes-npのリスナつくってるけどこれが残ってるせいで増え続けてたっぽい？onからonceに置き換えて修正しとこう
#NowPlaying
title:Marionetteは眠らない
artist:星井美希 (CV.長谷川明子)、伊吹翼 (CV.Machico)、北上麗花 (CV.平山笑美)、ジュリア (CV.寺川愛美)
album:THE IDOLM@STER LIVE THE@TER PERFORMANCE 06 - EP
#NowPlaying
title:Endless Journey
artist:チト(CV:水瀬いのり)、ユーリ(CV:久保ユリカ)
album:TVアニメ「少女終末旅行」オープニングテーマ「動く、動く」 - EP
AppleScriptのdataクラスがjsのバイナリ扱う型サポートしてくれてればなぁみたいなこと思う
@Cutls うまく動作してよかったです。npmパッケージ側がPRをマージorリジェクトしてObjC++モジュールを作成して新しいバージョンが公開されたらそれに対応させてPRを開きます
うごかねぇなぁって思ってたらdistにitunes.jsを移動させていないということに気づいたのでこれもPR送っとこう
別ブランチにcherry pickして新しくPR開いた。元のブランチはリセットしてマージしたものをforce push
iTunesの楽曲情報とかアートワークとかネイティブで取れればいいけどScripting Bridgeでしかとったことないのと、ネイティブなライブラリあるのかしらという気持ちになる。ITLibraryみたいなやつあったと思うけど使えんのかな
itunes-nowplaying-macに出してるPR、分けてやるべきことだな(´・ω・`)
ってpushしてから思うのダメでは
んー、prepareに置き換えたけどダメというかそもそも動いてない…？ってなってたけどgit指定したらnode_modules以下にディレクトリが存在してたらパッケージ更新されないの
@kPherox まって、Gitのglobal configでhttpsの接続とかをgitに置き換えたりするのにgithub:って接頭辞つけてるけどnpmにもそれ使われるの？やめて？
@Cutls npmパッケージの方でprepublishでビルドしている（お行儀が悪い形？）のでprepareに置き換えてみます
参考
gitリポジトリから取得したときに動くnpm scripts
https://qiita.com/k3nNy_51rcy/items/ee7d7351aa626283882c
prepareとprepublishの意図
https://qiita.com/ndxbn/items/f0cd2b13a3268254f2aa#npm-pack-したときに実行される-npm-script-の移り変わり
とりあえずitunes-nowplaying-macはprepareでビルドとindex.jsの移動、prepublishでdemo.jsを取り除くって形に変更してみよう
なんかnpm6でそんな風に変更されてるみたいだし…？
gitのprをnpmで指定したパッケージでprepublishないしprepare実行するって出来るのかしら
@Cutls ああ、distをpublishの時に作る形だからpull request指定だけじゃダメだった…
@kPherox 前回投稿のアカウントが別のアカウントの時にこうなるのを確認した
@kPherox 二つめのアカウントをデフォルトにしちゃうとこうなる？
アカウント間違えてあれって思ったら投稿のダイアログのアイコンが開いたときに変わってない
このアカウントは、notestockで公開設定になっていません。
