Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix #2403 アップロードファイルのAPIについてフルパスを取得する仕組み #2763

Closed
wants to merge 6 commits into from

Conversation

HungDV2022
Copy link
Collaborator

No description provided.

@ryuring
Copy link
Collaborator

ryuring commented Sep 27, 2023

@HungDV2022 これだと一覧の場合に取得できませんね。

ちょっとバーチャルフィールドについて調べたのですが、BlogPost エンティティに、次のコードを書けば、コントローラーで何もしなくても実装できそうです。

これは、eyecatch フィールドの値を変換するでなく、別に _eyecatch フィールドを追加する形になりますが、この仕様にしたいと思いますので、調整お願いします。

    protected $_virtual = ['_eyecatch'];
    protected function _get_eyecatch(){
        $BlogHelper = new BlogHelper(new View());
        return $BlogHelper->getEyeCatch($this, ['output' => 'url']);
    }

@ryuring ryuring assigned HungDV2022 and unassigned ryuring Sep 27, 2023
@ryuring ryuring added the Reviewed レビュー済 label Sep 27, 2023
@HungDV2022
Copy link
Collaborator Author

@ryuring いっぱい不要なファイルを入ったので別のブランチにPRします。
#2771

@HungDV2022 HungDV2022 closed this Sep 28, 2023
@gondoh gondoh added this to the 5.0.6 milestone Oct 19, 2023
@momofff momofff modified the milestones: 5.0.6, close Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Reviewed レビュー済
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants