In documentation it says “IFC4-CHANGE The entity has been deprecated and shall not be used. The entity IfcFacetedBrep shall be used instead.”
However FacetedBrep is a solid model. IfcFaceBasedSurfaceModel is not (necessarily). To me this seems incorrect
I agree, there is another problem that it is only deprecated by means of a textual reference, but not actually in the UML data model.
I guess IfcShellBasedSurfaceModel can be used instead of the FaceBased one. I never understood why we had both of them. They are considered the same shape representation type, where facetedbrep indeed is not.
Feel free to propose this change as a Pull request in GitHub - buildingSMART/IFC4.3.x-development: Repository to collect updates to the IFC4.3 Specification then we also have a reference to make the change to the UML data model.
[0] 8.18.6.3 IfcShapeRepresentationTypes - IFC 4.3.2 Documentation