HelloZeroNet / HelloZeroNet/ZeroNet

Nested content.jsons are not in sidebar.

オープン
#2,353 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
enhancement
主要言語
JavaScript
スター
18.8k
フォーク
2.3k
PR マージ指標
30日以内にマージされた PR はありません

説明

### Step 1: Please describe your environment

* ZeroNet version: 0.7.1 (rev4253)
* Operating system: Fedora 30
* Web browser: Tor Browser
* Tor status: always
* Opened port: no
* Special configuration: -

### Step 2: Describe the problem:

Nested `content.json`s are not listed in the sidebar. under the `CHOOSE` section.
#### Steps to reproduce:

A new blank zite, the root `content.json` includes `data/content.json` with `includes_allowed` set to true.
`data/content.json` includes `L_admin/content.json`.

#### Observed Results:

The sidebar will not show `L_admin/content.json` or any other nested `content.json` that is not explicitly in the includes entry of the root content.json.

The problematic line is in [SidebarPlugin.py line 504](https://github.com/HelloZeroNet/ZeroNet/blob/py3/plugins/Sidebar/SidebarPlugin.py#L504):

```
contents += list(site.content_manager.contents.get("content.json", {}).get("includes", {}).keys())
```
It does not consider nested includes.

**Additional context**
This issue is mirrored from the [GitCenter ZeroNet repository](http://127.0.0.1:43110/1GitLiXB6t5r8vuU2zC6a8GYj9ME6HMQ4t/repo/issues/view/?1P4w4Rvh4vS9r6G5xTiSh8qEW87EKZtnJB/11@1DfrA2M9Qra6stqT6tzwNMYNweswY54KAC)(you need to install ZeroNet to visit this link)

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

Start in plugins/Sidebar/SidebarPlugin.py around line 504 and inspect how the sidebar builds the CHOOSE list from content.json includes. Reproduce the nested data/content.json and L_admin/content.json setup described in the issue. Done means nested content.json files appear in the sidebar without being explicitly listed in the root includes entry.

索引モデルが issue の本文から書いたものです。

評価

技術スタック
python
領域
frontend
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。