Consider using a consistent type for diff_id throughout the PhabricatorPatch class

[suhaibmujahid]

Consider using a consistent type for diff_id throughout the class. Currently it accepts both str and int but the diff_id property always returns int, which could cause confusion.

Originally posted by @Copilot in #5303 (comment)