Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0045004 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] A. Platform | major | have not tried | 2020-09-07 13:30 | 2020-09-11 09:38 | |||
Reporter | shuehner | View Status | public | |||||
Assigned To | cberner | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | PR20Q4 | |||
Status | closed | Fix in branch | Fixed in SCM revision | |||||
Projection | none | ETA | none | Target Version | PR20Q4 | |||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Review Assigned To | ||||||||
Web browser | ||||||||
Modules | Core | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0045004: Remove obsolete old javascripts docs ui from client.kernel | |||||||
Description | The client.kernel contains a visualization of javascript code documentation using the jquery library which is accessible in any instance with this relative url: https://livebuilds.openbravo.com/erp_pi_pgsql/web/org.openbravo.client.kernel/docs/ [^] That is based on this 3rd party project: https://code.google.com/archive/p/code-illuminated/ [^] That was added long time ago and the syntax used for the documentation is very different from the 'jsdoc' syntax used today in i.e. modern retail javascript code. That old visualization should be deleted as obsolete. As extra benefit that removes an older version of the jquery library kept just for this. | |||||||
Steps To Reproduce | - | |||||||
Proposed Solution | 1.) Delete the whole org.openbravo.client.kernel/docs/ folder 2.) Also remove entry from legal/Licensing.txt referring to the 3rd party project used for this. 3.) Review and cleanup the org/openbravo/client/kernel/DocumentationComponent (and related constant) as it seems to be only used for enumerating js source files for this ? | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | |||||||||||||||
|
Notes | |
(0122874) hgbot (developer) 2020-09-09 13:04 |
Merge Request created: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/157 [^] |
(0122949) hgbot (developer) 2020-09-11 09:38 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/openbravo [^] Changeset: c2b45e21fe2ea0e429ee5ddb0727fdae55380856 Author: Cristian Berner <cristian.berner@openbravo.com> Date: 2020-09-11T07:38:26+00:00 URL: https://gitlab.com/openbravo/product/openbravo/-/commit/c2b45e21fe2ea0e429ee5ddb0727fdae55380856 [^] Fixes ISSUE-45004: Remove obsolete org.openbravo.client.kernel/docs Those docs were previously generated from javascript with code-illuminated library. The syntax is very different to the jsdoc used currently. So it is removed as obsolete, and both library and license for it are also removed. Template docs.js.ftl and class DocumentationComponent are also removed because they're not needed anymore by the module. --- M legal/Licensing.txt M modules/org.openbravo.client.kernel/src-db/database/sourcedata/OBCLKER_TEMPLATE.xml M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/KernelComponentProvider.java M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/KernelConstants.java R modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/DocumentationComponent.java R modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/templates/docs.js.ftl R modules/org.openbravo.client.kernel/web/org.openbravo.client.kernel/docs/docs/docs.css R modules/org.openbravo.client.kernel/web/org.openbravo.client.kernel/docs/docs/docs.js R modules/org.openbravo.client.kernel/web/org.openbravo.client.kernel/docs/index.html R modules/org.openbravo.client.kernel/web/org.openbravo.client.kernel/docs/scripts/jquery.dynacloud-5.js R modules/org.openbravo.client.kernel/web/org.openbravo.client.kernel/docs/scripts/jquery.js R modules/org.openbravo.client.kernel/web/org.openbravo.client.kernel/docs/scripts/prettify.css R modules/org.openbravo.client.kernel/web/org.openbravo.client.kernel/docs/scripts/prettify.js R modules/org.openbravo.client.kernel/web/org.openbravo.client.kernel/docs/scripts/wikicreole.js R modules/org.openbravo.client.kernel/web/org.openbravo.client.kernel/docs/tests/test_image_in_table.js R modules/org.openbravo.client.kernel/web/org.openbravo.client.kernel/docs/tests/test_with_cr.js R modules/org.openbravo.client.kernel/web/org.openbravo.client.kernel/docs/tests/test_with_crlf.js R modules/org.openbravo.client.kernel/web/org.openbravo.client.kernel/docs/tests/test_with_lf.js R modules/org.openbravo.client.kernel/web/org.openbravo.client.kernel/docs/tests/tests.html --- |
(0122950) hgbot (developer) 2020-09-11 09:38 |
Merge request merged: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/157 [^] |
Issue History | |||
Date Modified | Username | Field | Change |
2020-09-07 13:30 | shuehner | New Issue | |
2020-09-07 13:30 | shuehner | Assigned To | => platform |
2020-09-07 13:30 | shuehner | Modules | => Core |
2020-09-07 13:30 | shuehner | Triggers an Emergency Pack | => No |
2020-09-07 13:32 | shuehner | Description Updated | View Revisions |
2020-09-07 13:35 | shuehner | Proposed Solution updated | |
2020-09-07 13:48 | shuehner | Target Version | => PR20Q4 |
2020-09-09 13:04 | hgbot | Note Added: 0122874 | |
2020-09-09 17:52 | cberner | Assigned To | platform => cberner |
2020-09-10 14:17 | cberner | Relationship added | causes 0045028 |
2020-09-11 09:38 | hgbot | Resolution | open => fixed |
2020-09-11 09:38 | hgbot | Status | new => closed |
2020-09-11 09:38 | hgbot | Fixed in Version | => PR20Q4 |
2020-09-11 09:38 | hgbot | Note Added: 0122949 | |
2020-09-11 09:38 | hgbot | Note Added: 0122950 | |
2021-02-01 15:41 | cberner | Relationship added | related to 0045826 |
Copyright © 2000 - 2009 MantisBT Group |