silverdr

silverdr OP

While the same approach (i. e.) with view module as argument in Phoenix.View is not? Yes, I know it has been documented as being deprecated good time ago. I am asking about the reason for doing it in the first place and creating such difference/inconsistency on the way.

First 2 of 2 Posts Switch mode

al2o3cr

al2o3cr

The issue where the deprecation message was introduced has some discussion:

https://github.com/phoenixframework/phoenix/issues/2469#issuecomment-440340830

silverdr

silverdr OP

Thanks for the link. There’s:

I think our clauses were laid out in a way where you could pass a template where a view was expected or vice-versa because we were matching on the type and that can be a security issue if people are not careful. I mean, people would have to already be doing something wrong, but better to be safer and keep the API simple.

which (I think) tries to give an answer to the question, although I am afraid I don’t understand it properly enough to find it cogent. Not that it’s a major issue or so. Just, unless there’s an important reason for it, it feels like another tiny bit of unnecessary friction. It might only be me, but I tend to also value consistency, not only simplicity :wink: And potential security aspects could most probably be addressed without API changes.

Anyway, reworked all my calls to “render/4 with a view” in controllers into two calls in order to silence the deprecation warnings, and noted mentally down “no, no, in controllers you’re not supposed to do the same as in views!” :wink:

— All posts loaded —

Where Next? Top

Trending in Questions Top

stjefim
Hello! Suppose you are building workflow (order / task / payment) processing system with the following requirements: Each workflow con...
New
jonnycharles
I’m in search of an Elixir library that offers PDF generation capabilities similar to Ruby’s Prawn. While there have been discussions abo...
New
spammy
I’m looking to build a personal workflow to quickly deploy web applications written in elixir/phoenix, for local consumption (ie not on t...
New
dli
Before I dive in myself, did anyone successfully sprinkle Hologram into their existing LiveView app? Looking for hints regarding: Addi...
New
bottlenecked
Hi all, I wanted to ask how the community is dealing with post-release steps. Today we have Ecto migrations, which make sure that the db...
New
roeland
Kia ora, We have been using elixir-google-api to connect to Google Drive. However, with the updates to Tesla due to CVEs this is now bro...
New
rahultumpala
Hello, I have an Elixir backend that implements a custom protocol over TCP. I want to load test the backend and assess the performance o...
New

Other Trending Topics Top

JesseHerrick
Hey, I’m Jesse and I’m the main contributor behind Dexter, a full-featured, lightning-fast Elixir LSP optimized for large codebases. It s...
New
jimsynz
Beam Bots (or just BB for short) is a framework for building fault-tolerant robotics applications in Elixir using familiar OTP patterns. ...
New
Damirados
Hello everyone. After busy few months I am happy to announce v0.1.0 of Emerge & Solve. They are GUI (Emerge) and State management (S...
New
netoum
Corex is an accessible, unstyled UI component library for Phoenix that integrates Zag.js state machines using Vanilla JavaScript and Live...
New
ausimian
Emily is an Elixir library that runs Nx computations on Apple’s MLX. Install it as the default Nx backend and Nx, defn, Axon, Nx.Serving,...
New
juhalehtonen
There has been a thread to discuss the Stack Overflow Developer Survey on this forum every year since 2018, so here’s yet another one for...
New

We're in Beta

About us Mission Statement