replace import because the library randrange is used in line 101 and the library randint is not used

There is no change on the test behavior.

This functional test must follow the documentation, and this part
is not available in the documentation, then it must be removed.

So, we must use the setUp script to create the second person when
it is required.

Credential_updatePersonPassword is doing the same check, and it is proxied
script.

CredentialUpdate_updatePersonInformation is not proxied script, and accessing
getPassword requires a lot of privileges.

- do not alter default parameters when accessing them
- make explicit parameters take precedence over default,
  similarly to activate_kw

brown paper bag commit. Actually stop requiring skin_folder and use only skin_folder_id to unregister skin folders

Allow it to no longer be registered as well

VAT is now calculated in a completely different way.

To appease Chameleon

Modify Order_zSelectMovement  to make Packing List enable to represent Transfer of rights(ownership). To do this, remove a WHERE expression to exclude simulation movements that have same source and destination.

And Chameleon complains. Use tal:define instead.

Added new comments to show where some screenshots have been taken.

This allows ImageField to return original image content

Remove script because instead fix skins during the upgrade, it should be fixed and commited correctly in bt5

This makes tests independent and improve failures detection.

* Fix the issue that Sale Packing List wrongly has Root Applied Rule sometimes when it is created by Delivery Builder with a site which has many activity nodes.