Project

General

Profile

Bug(バグ) #3147

customizing.cssが生成された場合削除できない場合がある

Added by 開 石切山 about 7 years ago. Updated over 2 years ago.

Status:
Won't fix(対応せず)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2012-08-03
Due date:
% Done:

0%

3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Yes (はい)

Description

Overview (現象)

customizing.cssが生成された際アクセス権限が644のため削除できない場合がある。
SNSを削除する際customizing.cssの権限が足りずSNSを完全に削除できない。
customizing.cssが生成された際にも削除されるよう権限の付与が必要。

Environment (再現バージョン)

OpenPNE 3.6.4
OpenPNE 3.8.1

Way to repro (再現手順)

不明

Way to fix (修正内容)

web/cache/cssのパーミッションを777に
web/cache/css/customizing.cssのパーミッションを666に変更します。

備考

管理画面でデザインの設定はいじっていない。

$ ack "customizing.css" の結果より以下の部分が関係するか?

apps/pc_backend/modules/design/lib/opCustomCssForm.class.php
18:    $this->setDefault('css', Doctrine::getTable('SnsConfig')->get('customizing_css'));                                                                                          
25:    Doctrine::getTable('SnsConfig')->set('customizing_css', $this->getValue('css'));


Related issues

Related to OpenPNE 3 - Backport(バックポート) #3264: customizing.cssが生成された場合削除できない場合がある Fixed(完了) 2012-08-03
Related to OpenPNE 3 - Backport(バックポート) #3277: customizing.cssが生成された場合削除できない場合がある Fixed(完了) 2012-12-11

Associated revisions

Revision 9f7dbf23 (diff)
Added by kaoru n almost 7 years ago

(refs #3147)fixed to permission of css cache directory and file.

Revision c01b3f4d (diff)
Added by kaoru n over 6 years ago

(refs #3147)fixed to permission of css cache directory and file.

Revision 75b7b124 (diff)
Added by kaoru n over 6 years ago

(refs #3147) changed variable name csspath to cssPath.

Revision dd4fd05e
Added by Rimpei Ogawa over 6 years ago

Merge remote-tracking branch 'nishizoe/t3147note5' (refs #3147)

History

#1 Updated by Kousuke Ebihara almost 7 years ago

  • 3.8 で発生するか set to Yes (はい)

#2 Updated by kaoru n almost 7 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to kaoru n

#3 Updated by kaoru n over 6 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • Assignee deleted (kaoru n)
  • % Done changed from 0 to 50

https://github.com/openpne/OpenPNE3/pull/58
プルリクエストしました。

#4 Updated by Rimpei Ogawa over 6 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.9.0-old

こちらのチケットは 3.9.0 用のものとし、3.8用のチケットを別に作りました。 #3277

#5 Updated by Rimpei Ogawa over 6 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
  • Assignee set to kaoru n

master ブランチ向けの修正を別途作成お願いします。

https://github.com/openpne/OpenPNE3/pull/58 については、 #3277 でレビューします。

#6 Updated by kaoru n over 6 years ago

  • Description updated (diff)
  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

本文の修正内容を更新しました。

#7 Updated by kaoru n over 6 years ago

https://github.com/openpne/OpenPNE3/pull/67
プルリクエストしました。

#8 Updated by kaoru n over 6 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

#9 Updated by Rimpei Ogawa over 6 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

以下、修正してください。

  • css path は2語なので csspath ではなく cssPath とすべきです(コーディング規約)
  • 不要な空行が追加されています

https://github.com/openpne/OpenPNE3/pull/67 にも同様の内容をコメントしています。

#10 Updated by kaoru n over 6 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#11 Updated by kaoru n over 6 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

https://github.com/openpne/OpenPNE3/pull/67
修正しました。確認お願いします。

#12 Updated by Rimpei Ogawa over 6 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

修正確認しました。OKです。

#13 Updated by Rimpei Ogawa over 6 years ago

9f7dbf237b4880864be2bc8d6dfd127e28daa21e は本チケットに紐付いていますが、 #3277 に対する修正です。

#15 Updated by isao sano over 2 years ago

  • Status changed from Pending Testing(テスト待ち) to Won't fix(対応せず)
  • % Done changed from 70 to 0

OpenPNE 3.8.4 にて対応済みであったため、対応せずとします。

Also available in: Atom PDF