1. 12 Dec, 2019 5 commits
  2. 11 Dec, 2019 1 commit
    • Jérome Perrin's avatar
      CodingStyleTest: run testXHTML for each business template · 7988bc67
      Jérome Perrin authored
       - run `testXHTML` on business templates "one by one"
       - fix all the problems with the currently tested business templates
       - because `erp5_ingestion` was containing some tests skin folder depending on other business templates and because I did not want to add dependencies to `erp5_ingestion` I moved the tests in `erp5_ingestion_test`.
      
      /reviewed-on !989
      7988bc67
  3. 10 Dec, 2019 3 commits
  4. 06 Dec, 2019 4 commits
  5. 03 Dec, 2019 19 commits
    • Jérome Perrin's avatar
      CodingStyleTest: run testXHTML for each business template · 1bf425ca
      Jérome Perrin authored
      This will allow to find problems when only business templates and their
      dependencies are installed. When we run testXHTML with all business
      template installed we don't see problems that may happen only when some
      business templates are not installed.
      1bf425ca
    • Jérome Perrin's avatar
      accounting_simulation_legacy: format AccountingRuleCell properly · 9aa6b278
      Jérome Perrin authored
      pylint complained about indentation
      9aa6b278
    • Jérome Perrin's avatar
      accounting_simulation_legacy: restore AccountingRuleCell · 5d206c29
      Jérome Perrin authored
      this was removed in 2138aee5
      5d206c29
    • Jérome Perrin's avatar
      banking_inventory: skip coding style test · e8574779
      Jérome Perrin authored
      This bt has too many coding style problems for now
      e8574779
    • Jérome Perrin's avatar
      travel_expense: fix duplicate selection names · 03029bcb
      Jérome Perrin authored
        'ticket_view_selection'
      	'Tickect_viewRelatedRecordList/listbox': set(['erp5_hr_request'])
      	'Ticket_view/listbox': set(['erp5_crm'])
      03029bcb
    • Jérome Perrin's avatar
      officejs_ooffice: fix duplicate selection names · 2e272917
      Jérome Perrin authored
        'web_page_module_view_web_page_list_selection'
      	'DocumentModule_viewDocumentListAsJioForOofficeSpreadsheetEditor/listbox': set(['erp5_ooffice_spreadsheet_editor'])
      	'DocumentModule_viewDocumentListAsJioForOofficePresentationEditor/listbox': set(['erp5_ooffice_presentation_editor'])
      	'WebPageModule_viewWebPageList/listbox': set(['erp5_web'])
      	'DocumentModule_viewDocumentListAsJioForOofficeTextEditor/listbox': set(['erp5_ooffice_text_editor'])
      	'WebPageModule_viewWebTableListAsJioForWebTableEditor/listbox': set(['erp5_web_table_editor'])
      2e272917
    • Jérome Perrin's avatar
      officejs_jquery_app: fix duplicate selection name · e0591aeb
      Jérome Perrin authored
        'image_selection'
      	'ImageModule_viewImageList/listbox': set(['erp5_dms'])
      	'ImageModule_viewImageListAsJioForImageEditor/listbox': set(['erp5_image_editor'])
        'web_page_module_view_web_page_list_selection'
      	'DocumentModule_viewDocumentListAsJioForOofficeSpreadsheetEditor/listbox': set(['erp5_ooffice_spreadsheet_editor'])
      	'DocumentModule_viewDocumentListAsJioForOofficePresentationEditor/listbox': set(['erp5_ooffice_presentation_editor'])
      	'WebPageModule_viewWebPageList/listbox': set(['erp5_web'])
      	'DocumentModule_viewDocumentListAsJioForOofficeTextEditor/listbox': set(['erp5_ooffice_text_editor'])
      	'WebPageModule_viewWebTableListAsJioForWebTableEditor/listbox': set(['erp5_web_table_editor'])
      e0591aeb
    • Jérome Perrin's avatar
      research_item: fix missing selection name · 064a7095
      Jérome Perrin authored
      064a7095
    • Jérome Perrin's avatar
      tests: don't skip testLive* testVifib* · 0b951ab6
      Jérome Perrin authored
      There are no testVifib (at least not in nexedi/erp5 repository) and
      there is only one testLive that there is no reason to skip.
      0b951ab6
    • Jérome Perrin's avatar
      ingestion_test: fix pylint errors in testLiveIngestion · 58abc9d6
      Jérome Perrin authored
      This test never ran because we skip testLive*
      58abc9d6
    • Jérome Perrin's avatar
      ingestion: split erp5_ingestion in erp5_ingestion_test · 3fb72f4f
      Jérome Perrin authored
      erp5_ingestion contained a skin folder for tests, with some proxy fields depending on erp5_web.
      There was a proper test dependency to erp5_web, but in static analysis check we now check the skin folders, so it's no longer allowed to have some skin folder that are broken without test dependencies but not supposed to be used outside of tests.
      3fb72f4f
    • Jérome Perrin's avatar
      hr: fix proxy fields · 3487504f
      Jérome Perrin authored
      Since this was using proxy fields from erp5_crm, add dependency on CRM.
      Update to the new name of CRM field library.
      3487504f
    • Jérome Perrin's avatar
      video: fix <video> editor field · 16eb9ae3
      Jérome Perrin authored
      nuvalidator complains that:
        Bad value “true” for attribute “autoplay” on element “video”
      
      This could probably be switched to VideoField now.
      16eb9ae3
    • Jérome Perrin's avatar
      VideoField: make valid HTML5 regarding preload · 657bcc2e
      Jérome Perrin authored
      Unlike other attributes managed in video field, preload [1] is not a
      boolean attribute, so setting preload="preload" was invalid HTML5.
      
      On the field, we can only configure video_preload as a boolean
      attribute, setting preload="auto" when true and not setting preload when
      False seems the closest meaning of True/False.
      
      [1] https://html.spec.whatwg.org/multipage/media.html#attr-media-preload
      657bcc2e
    • Jérome Perrin's avatar
      tiosafe_core: depend on erp5_accounting · 8e99c75b
      Jérome Perrin authored
      IntegrationSite_view/my_source_payment is a proxyfield to a field in
      erp5_accounting.
      8e99c75b
    • Jérome Perrin's avatar
      post: depend on crm · 668d61df
      Jérome Perrin authored
      It uses proxy fields from erp5_crm
      668d61df
    • Jérome Perrin's avatar
      project_ui: fix duplicate selection names · 8169baee
      Jérome Perrin authored
         'base_view_document_selection'
      	'Base_viewDocumentList/listbox': set(['erp5_core'])
      	'Project_viewDocumentList/listbox': set(['erp5_web_project'])
         'project_module_selection'
      	'ProjectModule_viewProjectList/listbox': set(['erp5_project'])
      	'ProjectModule_viewProjectManagementList/listbox': set(['erp5_web_project'])
      8169baee
    • Jérome Perrin's avatar
      officejs: fix duplicate selection names · 4699cc45
      Jérome Perrin authored
      These selection names were already used in other business templates:
      
         'web_page_module_view_web_page_list_selection'
      	'WebPageModule_viewWebPageListAsJioForCodemirror/listbox': set(['erp5_officejs_codemirror'])
      	'WebPageModule_viewWebPageListAsJioForTextEditor/listbox': set(['erp5_text_editor'])
      	'WebPageModule_viewWebPageList/listbox': set(['erp5_web'])
      	'DocumentModule_viewDocumentListAsJioForPdfViewer/listbox': set(['erp5_officejs_pdf_viewer'])
         'image_selection'
      	'WebPageModule_viewWebPageListAsJioForSvgEditor/listbox': set(['erp5_officejs_svg_editor'])
      	'ImageModule_viewImageList/listbox': set(['erp5_dms'])
      4699cc45
    • Jérome Perrin's avatar
      monaco_editor_ui_test: repair test · 041b0614
      Jérome Perrin authored
      Now ERP5JS display list mode header only if the jio key does not contain
      /, so if we open a module with a hand-made URL with a trailing slash,
      it's no longer seen as a module.
      
      Repair this test by using a URL without that trailing / in the key.
      041b0614
  6. 02 Dec, 2019 8 commits