Acq is a complicated module and the new EDIFACT related options and features add another layer of complexity. It would be good if the feature could be turned off for libraries not interested in the functionality to help simplify working with acq a bit again.
Yes a 'use edifact' syspreference which if set to N (default) hides the edifact screens and links - which in fact are few
Hi Colin, EDIFACT appeared to be turnted on for me automatically. Are you sure this has been added? There is no mention of EDIFACT in the sysprefs.sql file:
I was also thinking about additional columns and buttons in other places - I haven't been able to get a total overview of the EDIFACT feature so far - but i seems there is something new on basket summary as well.
Hm, I found 134 ('EDIInvoicesShippingBudget',NULL,NULL,'The budget code used to allocate shipping charges to when processing EDI Invoice messages','free'), now - recheckign for EDI, but nothing else.
In most places if there are no edifact vendors present the functionality gets disabled. The main feature to control is probably the admin screens. A syspref shouldnt be to turn it off but to enable it - I'll see if I can generate a patch to add an enabling system preference
Created attachment 51185 [details] [review]
Patch to add the syspreference. Set to no the Edi related screens are hidden from the admin page. Also removes the edifact messages link in acq.
Created attachment 51228 [details] [review]
make an import explicit
Created attachment 54539 [details] [review]
Bug 16258 Add a syspref to govern EDIFACT functionality
Adds an "Edifact" systempreference to govern whether edifact processing
is enabled. In most places this is hidden if the current
vendor does not appear in the edi vendors table. This preference
hides the admin screens which define this and a couple of links.
Also fixes an anomaly whereby the basketgroup screen was not
making the same check on whether edi ordering should be enabled as
the basket screen. Both now use the same logic.
Rebase by firstname.lastname@example.org
Created attachment 54567 [details]
Button Create EDIFACT order still appears
If syspref "Edifact" is disabled but the vendor has an Edifact account, the button "Create EDIFACT order" still displays.
I would expect that it does no longer appear.
This patch works well, however there are some things that should be considered and improved before signing off on this:
1) With Edifact disabled, the EDI links still appear in the Admin sidebar navigation
2) With Edifact disabled, I can still directly navigate to edi_accounts.pl etc., either through the sidebar nav or going straight to the URL.
Patch no longer applies.