El problema es que vas haciendo cada replace "por turnos". Entonces primero reemplazas el 0 por 3. Pero más adelante reemplazas los 3 por 0, por lo que no obtienes el resultado buscado. De hecho, el cifrado de las letras tampoco funcionará correctamente, ya que por ejemplo primero cambias la "a" por "m", pero más adelante cambias la "m" por "j", por lo que al final las "aes" quedan como "jotas" en vez de "emes".

Lo que necesitas es una forma de hacer todos los cambios "a la vez" en un solo replace por así decir. Esta funcionalidad te la da `str.translate()`. Para usarlo tienes que crear antes una "tabla de traducción" mediante `str.maketrans()`.

Usando estas cosas, tu código se reduce a un par de líneas:

```
def encriptar(txt):
  cifra = str.maketrans("abcdefghijklmnopqrstuvwxyz0123456789", 
                        "mkwayxhpqfsdjtvrfuoiegcnlb3790285146")
  return txt.translate(cifra)
```

Y puedes verificar que funciona:

```
encriptar("1234567890")
```

Sale:

```
7902851463
```


## Bonus

La función desencriptar es análoga, intercambiando las cadenas usadas para crear la tabla de traducción.

```
def desencriptar(txt):
  cifra = str.maketrans("mkwayxhpqfsdjtvrfuoiegcnlb3790285146", 
                        "abcdefghijklmnopqrstuvwxyz0123456789")
  return txt.translate(cifra)
```