I see in the model account_analytic_line model the field 

'general_account_id': fields.many2one('account.account', 'General Account', required=True, ondelete='restrict'),

But why not have this a related field to the account_id field in account_move_line

'account_id': fields.many2one('account.account', 'Account', required=True, ondelete="cascade", domain=[('type','<>','view'),  ('type', '<>', 'closed')], select=2),

Changing the field than in account_move_line keeps the general_account_id in sync

Or is this a bad i