Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
5e63a3c
Merge pull request #1854 from RCOSDP/v0.9.26_hiroba_develop
kanon-yoneji May 8, 2026
982b35d
Merge pull request #1888 from RCOSDP/v0.9.26_hiroba_develop
kanon-yoneji Aug 3, 2026
5d24998
fix: charge process to use server-calculated price and handle price c…
nobuyoshikosaka Oct 19, 2026
b2f8c38
fix: get_search_data APIにitem-access権限チェックを追加
ivis-a-kondo Oct 19, 2026
3f10ebf
test: get_search_dataのACLテストを更新
ivis-a-kondo Oct 19, 2026
de8d08c
fix: update babel.ini to include additional Jinja2 extensions
nobuyoshikosaka Oct 19, 2026
2876e1f
fix: improve code readability and update charge confirmation messages
nobuyoshikosaka Oct 20, 2026
8c7874b
fix: include email modal strings in records UI catalog
nobuyoshikosaka Oct 20, 2026
cb273b7
test: get_search_dataのテスト期待値をログインユーザー除外仕様に変更
ivis-a-kondo Oct 20, 2026
5e52601
Convert PHP files to Python and update related configuration
ryoya-hayase Oct 23, 2026
5c0d9ec
Apply coding conventions
ryoya-hayase Oct 23, 2026
087401c
Fix redirect URLs in test cases to use .py extension instead of .php
ryoya-hayase Oct 23, 2026
0362a5b
fix: charge confirmation message to display previous price and unit
nobuyoshikosaka Oct 20, 2026
b0f3836
Merge pull request #1316 from ivis-a-kondo/ivis_a-kondo/v0.9.26_hiroba
kanon-yoneji Oct 23, 2026
c4182ab
Merge pull request #1314 from nobuyoshikosaka/hotfix-v0.9.26_hiroba_f…
kanon-yoneji Oct 23, 2026
8fbfe51
Merge pull request #1320 from ryoya-hayase/hotfix/W2026-64_hiroba_con…
kanon-yoneji Oct 23, 2026
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/invenio-accounts/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/invenio-communities/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/invenio-deposit/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/invenio-files-rest/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/invenio-mail/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/invenio-oaiharvester/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/invenio-oaiserver/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/invenio-oauth2server/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/invenio-previewer/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/invenio-records/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/invenio-resourcesyncclient/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/invenio-resourcesyncserver/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/weko-accounts/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
10 changes: 5 additions & 5 deletions modules/weko-accounts/tests/test_views.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,11 +55,11 @@ def test_redirect_method(app,mocker):
)
mock_render = mocker.patch("weko_accounts.views.redirect",return_value=make_response())
_redirect_method(False)
mock_render.assert_called_with("http://TEST_SERVER.localdomain/secure/login.php")
mock_render.assert_called_with("http://TEST_SERVER.localdomain/secure/login.py")

mock_render = mocker.patch("weko_accounts.views.redirect",return_value=make_response())
_redirect_method(True)
mock_render.assert_called_with("http://TEST_SERVER.localdomain/secure/login.php?next="+url)
mock_render.assert_called_with("http://TEST_SERVER.localdomain/secure/login.py?next="+url)

#def index():
# .tox/c1/bin/pytest --cov=weko_accounts tests/test_views.py::test_index -vv -s --cov-branch --cov-report=term --basetemp=/code/modules/weko-workflow/.tox/c1/tmp
Expand Down Expand Up @@ -398,15 +398,15 @@ def test_shib_stub_login(client,mocker):
# next is /
mock_redirect = mocker.patch("weko_accounts.views.redirect",return_value=make_response())
res = client.get(url+"?next=/")
mock_redirect.assert_called_with("http://test_server.localdomain/secure/login.php?next=/")
mock_redirect.assert_called_with("http://test_server.localdomain/secure/login.py?next=/")

# next is /next_page
res = client.get(url+"?next=/next_page")
mock_redirect.assert_called_with("http://test_server.localdomain/secure/login.php?next=/next_page")
mock_redirect.assert_called_with("http://test_server.localdomain/secure/login.py?next=/next_page")

# next is None
res = client.get(url)
mock_redirect.assert_called_with("http://test_server.localdomain/secure/login.php?next=/")
mock_redirect.assert_called_with("http://test_server.localdomain/secure/login.py?next=/")

current_app.config.update(
WEKO_ACCOUNTS_SHIB_IDP_LOGIN_ENABLED=False
Expand Down
2 changes: 1 addition & 1 deletion modules/weko-accounts/weko_accounts/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@
WEKO_ACCOUNTS_SHIB_LOGIN_CACHE_TTL = 180
"""Cache default timeout 3 minute"""

WEKO_ACCOUNTS_SHIB_IDP_LOGIN_URL = '{}secure/login.php'
WEKO_ACCOUNTS_SHIB_IDP_LOGIN_URL = '{}secure/login.py'
"""Login proxy URL."""

WEKO_ACCOUNTS_SSO_ATTRIBUTE_MAP = {
Expand Down
4 changes: 2 additions & 2 deletions modules/weko-admin/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
slient = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/weko-authors/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/weko-bulkupdate/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/weko-deposit/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/weko-gridlayout/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
4 changes: 2 additions & 2 deletions modules/weko-groups/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
slient = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/weko-handle/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/weko-index-tree/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/weko-indextree-journal/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/weko-items-autofill/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
4 changes: 2 additions & 2 deletions modules/weko-items-ui/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
slient = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
41 changes: 29 additions & 12 deletions modules/weko-items-ui/tests/test_views.py
Original file line number Diff line number Diff line change
Expand Up @@ -20469,25 +20469,44 @@ def test_index_upload_acl(client, db_sessionlifetime, users, id, status_code):

# def get_search_data(data_type=''):
# .tox/c1/bin/pytest --cov=weko_items_ui tests/test_views.py::test_get_search_data_acl -vv -s --cov-branch --cov-report=term --basetemp=/code/modules/weko-items-ui/.tox/c1/tmp
def test_get_search_data_acl(client_api, db_sessionlifetime):
def test_get_search_data_acl_nologin(client_api, db_sessionlifetime):
url = url_for(
"weko_items_ui_api.get_search_data", data_type="username", _external=True
)
res = client_api.get(url)
assert res.status_code == 200
url = url_for(
"weko_items_ui_api.get_search_data", data_type="email", _external=True
)
res = client_api.get(url)
assert res.status_code == 200
assert res.status_code == 302

url = url_for("weko_items_ui_api.get_search_data", data_type="hoge", _external=True)
res = client_api.get(url)
assert res.status_code == 200

@pytest.mark.parametrize(
"id, status_code",
[
(0, 200),
(1, 200),
(2, 200),
(3, 200),
(4, 403),
(5, 403),
(6, 200),
(7, 403),
],
)
def test_get_search_data_acl(client_api, db_sessionlifetime, users, id, status_code):
login_user_via_session(client=client_api, email=users[id]["email"])

for data_type in ("username", "email", "hoge"):
url = url_for(
"weko_items_ui_api.get_search_data",
data_type=data_type,
_external=True,
)
res = client_api.get(url)
assert res.status_code == status_code


# .tox/c1/bin/pytest --cov=weko_items_ui tests/test_views.py::test_get_search_data -vv -s --cov-branch --cov-report=term --basetemp=/code/modules/weko-items-ui/.tox/c1/tmp
def test_get_search_data(client_api, users, db_userprofile, db_sessionlifetime):
login_user_via_session(client=client_api, email=users[0]["email"])

url = url_for(
"weko_items_ui_api.get_search_data", data_type="username", _external=True
)
Expand All @@ -20497,7 +20516,6 @@ def test_get_search_data(client_api, users, db_userprofile, db_sessionlifetime):
"error": "",
"results": [
"user",
"contributor",
"comadmin",
"repoadmin",
"sysadmin",
Expand All @@ -20515,7 +20533,6 @@ def test_get_search_data(client_api, users, db_userprofile, db_sessionlifetime):
"error": "",
"results": [
"user@test.org",
"contributor@test.org",
"comadmin@test.org",
"repoadmin@test.org",
"sysadmin@test.org",
Expand Down
1 change: 1 addition & 0 deletions modules/weko-items-ui/weko_items_ui/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -671,6 +671,7 @@ def index_upload():

@blueprint_api.route('/get_search_data/<data_type>', methods=['GET'])
@login_required
@item_permission.require(http_exception=403)
def get_search_data(data_type=''):
"""get_search_data.

Expand Down
3 changes: 2 additions & 1 deletion modules/weko-itemtypes-ui/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
3 changes: 2 additions & 1 deletion modules/weko-logging/babel.ini
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ encoding = utf-8

[jinja2: **/templates/**.html]
encoding = utf-8
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension
silent = false
extensions = jinja2.ext.autoescape, jinja2.ext.with_, webassets.ext.jinja2.AssetsExtension, jinja2.ext.loopcontrols, jinja2.ext.i18n, jinja2.ext.do

# Extraction from JavaScript files

Expand Down
Loading
Loading